Skip to content

Include already existing JSON operators in other postgres adapters and fix contains for bigint[] #731

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Include already existing JSON operators in other postgres adapters and fix contains for bigint[] #731

wants to merge 2 commits into from

Conversation

jgrazina
Copy link
Contributor

@jgrazina jgrazina commented Apr 10, 2025 •
edited
Loading

  • Multiple postgres adapters were supporting JSON/JSONB columns, but were missing out on JSON operators defined in PostgreDialectJSON dialect
  • Contains for bigint[] was using ilike incorrecly

@jgrazina jgrazina changed the title Include already existing JSON operators in other postgres adapters Include already existing JSON operators in other postgres adapters and fix contains for bigint[] Apr 11, 2025
@leonelcamara
Copy link
Contributor

@mdipierro I can't really use the postgres3 adapter without this change, since its contains is bugged and it doesn't have the json operator, should be merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jgrazina @leonelcamara