Skip to content

[CALCITE-7512] Support array operators for PostgreSQL#4930

Open
cjj2010 wants to merge 3 commits into
apache:mainfrom
cjj2010:CALCITE-7512
Open

[CALCITE-7512] Support array operators for PostgreSQL#4930
cjj2010 wants to merge 3 commits into
apache:mainfrom
cjj2010:CALCITE-7512

Conversation

@cjj2010

@cjj2010 cjj2010 commented May 9, 2026

Copy link
Copy Markdown
Contributor

@sonarqubecloud

Copy link
Copy Markdown

* PostgreSQL JSON Functions</a>
*/
@LibraryOperator(libraries = {POSTGRESQL})
public static final SqlBinaryOperator CONTAINS_OP =

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

You take types ANY_ANY, which means that this will validate for non-array types, or for array types that have different element types. I think the type checking needs to be improved.

@mihaibudiu mihaibudiu added the returned-with-feedback There are review comments (in JIRA and/or in GitHub) to be implemented before merging the PR label Jun 1, 2026
@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown

This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 90 days if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@calcite.apache.org list. Thank you for your contributions.

@github-actions github-actions Bot added the stale label Jul 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

returned-with-feedback There are review comments (in JIRA and/or in GitHub) to be implemented before merging the PR stale

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants