[PR] Fix nondeterministic age_global_graph regression test [age]

["jrgemignani (via GitHub)" <gi...@apache.org>]

jrgemignani opened a new pull request, #2365:
URL: https://github.com/apache/age/pull/2365

   The age_global_graph test had two issues that could cause intermittent failures:
   
   1. Nondeterministic warning output: The graph_stats() call on a graph with deliberately deleted vertices produces WARNING messages for dangling edges. These warnings are emitted by iterating edge label tables (knows, stalks), and the iteration order is not guaranteed. Since PostgreSQL WARNING messages cannot be caught or counted from SQL (only ERROR and above are catchable via PL/pgSQL exception handling), we suppress them with SET client_min_messages = error. The suppressed warnings are documented verbatim in comments. The graph_stats() result row still validates correct dangling-edge handling.
   
   2. Nondeterministic row ordering: Multiple MATCH...RETURN queries returned multi-row results without ORDER BY, relying on scan order. Added ORDER BY id(u), id(v), id(e), id(n), or id(a) as appropriate to all MATCH...RETURN queries for future-proofing, even those currently returning a single row.
   
   Files changed:
     regress/sql/age_global_graph.sql
     regress/expected/age_global_graph.out


-- 
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: commits-unsubscribe@age.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] Fix nondeterministic age_global_graph regression test [age]

["MuhammadTahaNaveed (via GitHub)" <gi...@apache.org>]

MuhammadTahaNaveed merged PR #2365:
URL: https://github.com/apache/age/pull/2365


-- 
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: commits-unsubscribe@age.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] Fix nondeterministic age_global_graph regression test [age]

["tcolo (via GitHub)" <gi...@apache.org>]

tcolo commented on PR #2365:
URL: https://github.com/apache/age/pull/2365#issuecomment-4189162522

   @jrgemignani I reviewed the patch it looks fine to me - claude code didn't find anything and also compile and make install works on macosx without problems.


-- 
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: commits-unsubscribe@age.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org