Skip to content

Conversation

@PsiACE
Copy link
Member

@PsiACE PsiACE commented Aug 14, 2024

Which issue does this PR close?

Closes #.

Rationale for this change

Part of #11752 and #11790

What changes are included in this PR?

Are these changes tested?

YES, with sqllogictests.

Are there any user-facing changes?

NO

Signed-off-by: Chojan Shang <psiace@apache.org>
@github-actions github-actions bot added sqllogictest SQL Logic Tests (.slt) functions Changes to functions implementation labels Aug 14, 2024
Copy link
Contributor

@alamb alamb left a comment

Choose a reason for hiding this comment

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

👏 -- very nice -- thank you @PsiACE

200w

@alamb alamb merged commit a00922b into apache:main Aug 14, 2024
@PsiACE PsiACE deleted the substrindex branch August 15, 2024 00:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

functions Changes to functions implementation sqllogictest SQL Logic Tests (.slt)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants