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

feat: support array containment @>, <@ #13180

Closed
Tracked by #7117
thexiay opened this issue Oct 31, 2023 · 4 comments
Closed
Tracked by #7117

feat: support array containment @>, <@ #13180

thexiay opened this issue Oct 31, 2023 · 4 comments

Comments

@thexiay
Copy link
Contributor

thexiay commented Oct 31, 2023

Is your feature request related to a problem? Please describe.

This is a subfunction of #7117

This is appropriate as my good first issue

Describe the solution you'd like

No response

Describe alternatives you've considered

No response

Additional context

No response

@github-actions github-actions bot added this to the release-1.5 milestone Oct 31, 2023
@thexiay thexiay changed the title feat: support array containment feat: support array containment @>, <@ Oct 31, 2023
@thexiay thexiay changed the title feat: support array containment @>, <@ feat: support array containment @>, <@ Oct 31, 2023
@thexiay
Copy link
Contributor Author

thexiay commented Nov 4, 2023

Infact pgsql has type range but risingwave does not have type defined range , but this operator usage like this:
image
Is there necessary to add array contains element ? I try it in pgsql and it;s not ok:
image
Or Just Keep the behavior consistent with pgsql?
cc @wangrunji0408 @st1page @lmatz

@wangrunji0408
Copy link
Contributor

Hi. We'll keep consistent with pg and we are going to add array @> array soon. This should be able to support array containment. There will be no array contains element operation for this.

@thexiay
Copy link
Contributor Author

thexiay commented Nov 4, 2023

Hi. We'll keep consistent with pg and we are going to add array @> array soon. This should be able to support array containment. There will be no array contains element operation for this.

I have done this issue and I will request a mr as soon as possible.

thexiay pushed a commit to thexiay/risingwave that referenced this issue Nov 5, 2023
github-merge-queue bot pushed a commit that referenced this issue Nov 7, 2023
@stdrc
Copy link
Member

stdrc commented Nov 7, 2023

Closed by #13253

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

No branches or pull requests

3 participants