You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Fixes EntityEdge.reference_time being lost when edges are read from the database.
Although reference_time is persisted on entity edges, it was omitted from the
Cypher return projections. As a result, edge hydration always received a missing
value and set reference_time to None.
This PR:
Adds reference_time to the shared entity-edge return projection.
Adds it to the standalone Neptune full-text, similarity, and BFS search
projections.
Adds regression tests for query projection and edge hydration.
Type of Change
Bug fix
New feature
Performance improvement
Documentation/Tests
Objective
Ensure persisted EntityEdge.reference_time values are preserved across standard
edge reads and search results.
Testing
Unit tests added/updated
Integration tests added/updated
All existing tests pass
Regression tests verify that:
Entity-edge return queries select reference_time for every graph provider.
Both entity-edge hydration paths preserve the returned timestamp.
The full test suite was not run locally due to environment limitations.
Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. For privacy information, see our Privacy Notice. You can sign the CLA by just posting a Pull Request Comment same as the below format.
I have read the CLA Document and I hereby sign the CLA behalf on myself, e-mail: example@example.com
or
I have read the CLA Document and I hereby sign the CLA behalf of my company, e-mail: example@example.com
Signature is valid for 6 months.
This bot will be retriggered when the Contributor License Agreement comment has been provided. Posted by the CLA Assistant Lite bot.
I have read the CLA Document and I hereby sign the CLA behalf on myself, e-mail: sakshigchitnis27@gmail.com
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fixes
EntityEdge.reference_timebeing lost when edges are read from the database.Although
reference_timeis persisted on entity edges, it was omitted from theCypher return projections. As a result, edge hydration always received a missing
value and set
reference_timetoNone.This PR:
reference_timeto the shared entity-edge return projection.projections.
Type of Change
Objective
Ensure persisted
EntityEdge.reference_timevalues are preserved across standardedge reads and search results.
Testing
Regression tests verify that:
reference_timefor every graph provider.The full test suite was not run locally due to environment limitations.
Breaking Changes
No breaking changes.
Checklist
make lintpasses)No documentation changes were necessary because this restores the existing model
field’s expected read behavior.
Related Issues
Closes #1661