Copilot commented on code in PR #2550:
URL: https://github.com/apache/age/pull/2550#discussion_r3846053725
##########
regress/sql/cypher_vle.sql:
##########
@@ -487,6 +487,26 @@ $$) AS (person agtype, friend agtype);
SELECT drop_graph('issue_2382', true);
+--
+-- Issue #2549: VLE path + DETACH DELETE + RETURN p should not raise
+-- stale TID error. When the path's bound vertices are deleted before
+-- the path datum is materialized, build_path must tolerate the stale
+-- TID and return the path with empty properties ({}) for the deleted
+-- entities.
Review Comment:
The new test comment says the deleted entities should return with empty
properties ({}), but the expected output below shows some deleted vertices
still have their original properties (e.g., {"id": 1}/{"id": 2}/{"id": 3}).
Consider rewording the comment to match the actual behavior being asserted
here: projection should not error, and any vertex/edge whose stored TID is no
longer visible should materialize with empty properties ({}).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
