Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[NEOS-1742] Add integration test that ensures PG create statements don't return dupes #3245

Open
nickzelei opened this issue Feb 11, 2025 · 0 comments
Labels
bug Something isn't working Fast Follow Created by Linear-GitHub Sync Postgresql Created by Linear-GitHub Sync
Milestone

Comments

@nickzelei
Copy link
Member

nickzelei commented Feb 11, 2025

We discovered an issue where a create table statement returned duplicate columns.

This was caused by a serial value returning a non-sequence record in our linked sequence CTE.

This happened because of an index that was composite and referenced the column.

We fixed it but should add a test.

Might also be worth handling the de-duping at the API layer too?

The PR of the original fix is here: #3244

From SyncLinear.com | NEOS-1742

@nickzelei nickzelei added bug Something isn't working Fast Follow Created by Linear-GitHub Sync Postgresql Created by Linear-GitHub Sync labels Feb 11, 2025
@nickzelei nickzelei added this to the v.40 milestone Feb 24, 2025
@nickzelei nickzelei modified the milestones: v.40, v.41 Mar 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Fast Follow Created by Linear-GitHub Sync Postgresql Created by Linear-GitHub Sync
Projects
None yet
Development

No branches or pull requests

1 participant