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.

Sign up for GitHub

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

Jump to bottom

Add SQL REGEXP_EXTRACT function; add "index" to "regex" extractionFn. #4055

Merged
merged 2 commits into from
Mar 25, 2017
Merged

Add SQL REGEXP_EXTRACT function; add "index" to "regex" extractionFn. #4055

merged 2 commits into from
Mar 25, 2017

Conversation

gianm
Copy link
Contributor

@gianm gianm commented Mar 14, 2017

Syntax is the same as Hive/Presto's REGEXP_EXTRACT.

@gianm gianm added the Feature label Mar 14, 2017
@gianm gianm added this to the 0.10.1 milestone Mar 14, 2017
@jihoonson
Copy link
Contributor

LGTM!

@fjy
Copy link
Contributor

fjy commented Mar 24, 2017

👍

1 similar comment
@jon-wei
Copy link
Contributor

jon-wei commented Mar 25, 2017

👍

@jon-wei jon-wei merged commit dd6c0ab into apache:master Mar 25, 2017
@jon-wei jon-wei mentioned this pull request Jun 13, 2017
Closed
seoeun25 pushed a commit to seoeun25/incubator-druid that referenced this pull request Jan 10, 2020
@navis
Backport of apache#4055 (add "index" to "regex" extractionFn)
dc709a2
@gianm gianm deleted the sql-regex branch September 23, 2022 19:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Feature
Projects
None yet
Milestone
0.10.1
Development

Successfully merging this pull request may close these issues.
4 participants
@gianm @jihoonson @fjy @jon-wei