You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@age.apache.org by "crdv7 (via GitHub)" <gi...@apache.org> on 2026/08/31 08:53:13 UTC
[PR] Fix locale-dependent ordering in cypher_merge tests [age]
crdv7 opened a new pull request, #2558:
URL: https://github.com/apache/age/pull/2558
The `issue_1446` regression test orders `labels(n)` inside Cypher. Since the
result is an agtype value, its string components are compared using the
database's default collation. This makes the output order locale-dependent.
For example, the unmodified test returns the mixed-case labels in the
following orders:
```text
C: X, Y, Z, hub, shared
en_US.utf8: hub, shared, X, Y, Z
```
The entities and counts are correct in both cases, but `pg_regress` reports a
failure because the row order differs from the expected output.
Move the incidental ordering out of Cypher and sort the displayed label text
with an explicit `C` collation. Apply the same deterministic ordering to both
`issue_1446` verification queries. The test continues to verify the same
MERGE results without depending on the database locale.
This is the same class of regression-test portability issue addressed by
[#2439](https://github.com/apache/age/pull/2439).
--
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 locale-dependent ordering in cypher_merge tests [age]
Posted by "MuhammadTahaNaveed (via GitHub)" <gi...@apache.org>.
MuhammadTahaNaveed closed pull request #2558: Fix locale-dependent ordering in cypher_merge tests
URL: https://github.com/apache/age/pull/2558
--
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 locale-dependent ordering in cypher_merge tests [age]
Posted by "MuhammadTahaNaveed (via GitHub)" <gi...@apache.org>.
MuhammadTahaNaveed merged PR #2558:
URL: https://github.com/apache/age/pull/2558
--
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