Skip to content

Conversation

gvolfing
Copy link
Contributor

@gvolfing gvolfing commented Mar 11, 2024

Memgraph currently provides id based lookup for vertices, but lacks to ability to do the same for edge ids.
This PR is resolving that issue.

Resolves #1772

@gvolfing gvolfing self-assigned this Mar 11, 2024
@gvolfing gvolfing added this to the mg-v2.16.0 milestone Mar 11, 2024
@gvolfing gvolfing added the Docs needed Docs needed label Mar 11, 2024
@gvolfing gvolfing force-pushed the Fetch-edge-by-edge-id branch from 99465e8 to 9665117 Compare March 15, 2024 07:21
@gvolfing gvolfing mentioned this pull request Mar 15, 2024
4 tasks
@gvolfing gvolfing marked this pull request as ready for review March 18, 2024 12:59
Copy link
Contributor

@antoniofilipovic antoniofilipovic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a few comments. This ended up as a nice solution 💪

Copy link
Contributor

@Darych Darych left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@gvolfing gvolfing merged commit 2b5c04a into master Mar 27, 2024
@gvolfing gvolfing deleted the Fetch-edge-by-edge-id branch March 27, 2024 11:38
@kgolubic
Copy link
Contributor

kgolubic commented Apr 7, 2024

@gvolfing can you please provide me with Release notes?

I would go with: introduced the ability to fetch edges based on edge ID

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Docs needed Docs needed

4 participants