Skip to content
This repository has been archived by the owner on Sep 18, 2023. It is now read-only.

[NSE-818] Support length, char_length, locate & regexp_extract #847

Merged
merged 9 commits into from
Apr 21, 2022

Conversation

PHILO-HE
Copy link
Collaborator

What changes were proposed in this pull request?

Support expressions like length, char_length, locate, regexp_extract.

How was this patch tested?

Some spark unit tests covers these expressions. A unit test suite is added in gandiva for testing regexp_extract.

@github-actions
Copy link

#818

@PHILO-HE PHILO-HE changed the title [NSE-818] Support length, char_length, locate, regexp_extract [NSE-818] Support length, char_length, locate & regexp_extract Apr 14, 2022
@PHILO-HE
Copy link
Collaborator Author

This PR depends on oap-project/arrow#96

@zhouyuan
Copy link
Collaborator

@PHILO-HE can you do a rebase?

@PHILO-HE PHILO-HE force-pushed the regexp_extract branch 2 times, most recently from d879672 to 5bb5d30 Compare April 19, 2022 05:53
@PHILO-HE
Copy link
Collaborator Author

Changed the dependent PR to oap-project/arrow#97 for arrow-4.0. Test passed.

@zhouyuan zhouyuan merged commit 913f7ae into oap-project:master Apr 21, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants