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

   age_global_graph: stabilize regression tests under concurrent xid load
   
   Wrap both vertex_stats() context-building phases in a single BEGIN ISOLATION 
LEVEL REPEATABLE READ; ... COMMIT; transaction so the three calls share one 
snapshot. This prevents the snapshot-fallback path in is_ggctx_invalid() from 
purging an already-built graph context when concurrent xid activity 
(autovacuum, parallel installcheck, replication, shared CI) advances the 
snapshot between calls, which would otherwise make the targeted 
delete_global_graphs(name) checks return false instead of the expected true. 
Read Committed is insufficient because it acquires a fresh snapshot per 
statement; REPEATABLE READ pins one snapshot for the whole transaction.
   
   Also add explicit ORDER BY id to the three direct-SQL label-table SELECTs 
(_ag_label_vertex x2, _ag_label_edge) that return multiple rows, so their 
output no longer depends on heap scan order.
   
   This is a test-only change (regress/sql/age_global_graph.sql and 
regress/expected/age_global_graph.out); no extension C code or SQL is modified.
   
   All 37 regression tests pass (installcheck) on PostgreSQL 18.3.
   
   Co-authored-by: GitHub Copilot <[email protected]>
   
   modified:   regress/expected/age_global_graph.out
   modified:   regress/sql/age_global_graph.sql


-- 
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]

Reply via email to