Skip to content

nix: add postgrest-with-oriole-17#4845

Draft
wolfgangwalther wants to merge 1 commit intoPostgREST:mainfrom
wolfgangwalther:with-orioledb
Draft

nix: add postgrest-with-oriole-17#4845
wolfgangwalther wants to merge 1 commit intoPostgREST:mainfrom
wolfgangwalther:with-orioledb

Conversation

@wolfgangwalther
Copy link
Copy Markdown
Member

@wolfgangwalther wolfgangwalther commented Apr 27, 2026

Tests PostgREST against orioledb, which is packaged on a temporary branch on my Nixpkgs fork. Thus, not ready for merging, yet.

We will be able to see some failing spec tests, though - did not check the other test suites, yet.

Comment on lines -3243 to 3245
create index bets_data_json on bets ((data_json ->>'contractId'));
-- TODO orioledb: Could not identify a comparison function for type json
-- create index bets_data_json on bets ((data_json ->>'contractId'));
create index bets_data_jsonb on bets ((data_jsonb ->>'contractId'));
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is also worth noting, had to uncomment that line, otherwise loading the fixtures would have errored out. Some test failures are possibly related to this.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like this causes quite a few spec test failures, judging by the regular PG version spec tests failing. So this should probably be fixed first to see what else remains.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reported the bug on orioledb/orioledb#836

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can repro locally all the oriole failed tests on CI.

Finished in 63.4601 seconds
1315 examples, 57 failures, 1 pending

Opened another obvious bug on orioledb/orioledb#837, it looks there are others but they would need to solve orioledb/orioledb#836 first otherwise it's hard to pinpoint.

@steve-chavez
Copy link
Copy Markdown
Member

Can repro the same results as on CI locally: https://github.com/PostgREST/postgrest/actions/runs/24987776179/job/73165340783?pr=4845

1315 examples, 57 failures, 1 pending

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants