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

fix argument type of prefix_in #16016

Merged
merged 2 commits into from
May 11, 2024
Merged

fix argument type of prefix_in #16016

merged 2 commits into from
May 11, 2024

Conversation

aunjgr
Copy link
Contributor

@aunjgr aunjgr commented May 11, 2024

What type of PR is this?

  • API-change
  • BUG
  • Improvement
  • Documentation
  • Feature
  • Test and CI
  • Code Refactoring

Which issue(s) this PR fixes:

issue #16009

What this PR does / why we need it:

change type of prefix_in's second argument from T_tuple to T_varchar

@matrix-meow matrix-meow added the size/XS Denotes a PR that changes [1, 9] lines label May 11, 2024
@mergify mergify bot added the kind/bug Something isn't working label May 11, 2024
@aunjgr aunjgr changed the title fix type signature of prefix_in fix argument type of prefix_in May 11, 2024
@matrixorigin matrixorigin deleted a comment from matrix-meow May 11, 2024
@mergify mergify bot merged commit 3485cc0 into matrixorigin:main May 11, 2024
18 of 19 checks passed
aunjgr added a commit to aunjgr/matrixone that referenced this pull request May 11, 2024
change type of prefix_in's second argument from T_tuple to T_varchar

Approved by: @m-schen, @badboynt1, @ouyuanning
aunjgr added a commit to aunjgr/matrixone that referenced this pull request May 11, 2024
change type of prefix_in's second argument from T_tuple to T_varchar

Approved by: @m-schen, @badboynt1, @ouyuanning
@aunjgr aunjgr deleted the prefix_in branch May 17, 2024 03:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Something isn't working size/XS Denotes a PR that changes [1, 9] lines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants