[PR] resolve subgraph staging sequences via regclass [age]

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

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

   The vertex/edge staging copies in create_subgraph() generated new graphids with nextval(%L), which binds the sequence as a string literal and invokes the nextval(text) overload. That re-resolves the schema-qualified sequence name on each call.
   
   Cast the literal to regclass (nextval(%L::regclass)) so the sequence is resolved once to its OID, matching how AGE defines its label id defaults (nextval('schema.seq'::regclass)). Applied to both the vertex and edge staging queries, in sql/age_subgraph.sql and the identical body in the age--1.7.0--y.y.y.sql upgrade template so the upgrade-path catalog comparison still matches.
   
   Behavior is unchanged; all 38 regression tests pass against PostgreSQL 18.
   
   Addresses Copilot review feedback on apache/age#2441.
   
   Co-authored-by: GitHub Copilot (Claude Opus 4.8) <[email protected]>
   
   modified:   age--1.7.0--y.y.y.sql
   modified:   sql/age_subgraph.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: commits-unsubscribe@age.apache.org

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

Re: [PR] resolve subgraph staging sequences via regclass [age]

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

gregfelice merged PR #2446:
URL: https://github.com/apache/age/pull/2446


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