Skip to content

Fix: VectorIndexId to mgp_value type conversion#3767

Merged
DavIvek merged 3 commits intomasterfrom
fix-vector-index-mgp-value-type-conversion
Feb 9, 2026
Merged

Fix: VectorIndexId to mgp_value type conversion#3767
DavIvek merged 3 commits intomasterfrom
fix-vector-index-mgp-value-type-conversion

Conversation

@DavIvek
Copy link
Copy Markdown
Contributor

@DavIvek DavIvek commented Feb 9, 2026 •
edited
Loading

This PR fixes VectorIndexId to mgp_value type conversion.
closes #3765

@DavIvek DavIvek self-assigned this Feb 9, 2026
@DavIvek
Copy link
Copy Markdown
Contributor Author

DavIvek commented Feb 9, 2026 •
edited
Loading

Tracking

  • [Link to Epic/Issue]

Standard development

CI Testing Labels

  • Select the appropriate CI test labels (CI -build=build-name -test=test-suite)

Documentation checklist

  • Add the documentation label
  • Add the bug / feature label
  • Add the milestone for which this feature is intended
    • If not known, set for a later milestone
  • Write a release note, including added/changed clauses
    • This PR fixes the conversion of VectorIndexId to mgp_value. As a result, an exception is no longer thrown when reading vector index properties from query modules. #3767
  • [ Documentation PR link memgraph/documentation#XXXX ]
    • Is back linked to this development PR

@DavIvek DavIvek added this to the mg-v3.8.0 milestone Feb 9, 2026
@DavIvek DavIvek added bug bug Docs - changelog only Docs - changelog only CI -build=release -test=e2e Run release build and e2e tests on push CI -build=release -test=core Run release build and core tests on push labels Feb 9, 2026
@DavIvek DavIvek marked this pull request as ready for review February 9, 2026 10:45
@DavIvek DavIvek requested a review from imilinovic February 9, 2026 10:45
@DavIvek DavIvek changed the title Fix VectorIndexId to mgp_value type conversion Fix: VectorIndexId to mgp_value type conversion Feb 9, 2026
@DavIvek DavIvek requested a review from colinbarry February 9, 2026 11:07
@DavIvek DavIvek enabled auto-merge February 9, 2026 11:27
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Feb 9, 2026

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@DavIvek DavIvek added this pull request to the merge queue Feb 9, 2026
Merged via the queue into master with commit fc5b262 Feb 9, 2026
40 checks passed
@DavIvek DavIvek deleted the fix-vector-index-mgp-value-type-conversion branch February 9, 2026 13:23
@gitbuda gitbuda mentioned this pull request Feb 10, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@imilinovic imilinovic imilinovic approved these changes

@colinbarry colinbarry Awaiting requested review from colinbarry

Assignees

@DavIvek DavIvek

Labels

bug bug CI -build=release -test=core Run release build and core tests on push CI -build=release -test=e2e Run release build and e2e tests on push Docs - changelog only Docs - changelog only

Projects

None yet

Milestone

mg-v3.8.0

Development

Successfully merging this pull request may close these issues.

Failure of subsequent embeddings.node_sentence call

2 participants

@DavIvek @imilinovic