Skip to content

[docs] Document the optional group index of regexp_extract_all - #4022

Draft
ccl125 wants to merge 1 commit into
apache:masterfrom
ccl125:docs/regexp-extract-all-group-index
Draft

ccl125 wants to merge 1 commit into
apache:masterfrom
ccl125:docs/regexp-extract-all-group-index

Conversation

@ccl125

@ccl125 ccl125 commented Jul 30, 2026

Copy link
Copy Markdown

What changed

Document the optional third parameter <idx> of REGEXP_EXTRACT_ALL, covering:

  • Updated syntax: REGEXP_EXTRACT_ALL(<str>, <pattern>[, <idx>])
  • New <idx> row in the parameters table
  • A new "Extracting a specific capturing group" section with examples: idx = 0 (whole match), idx = N (N-th capturing group), empty strings for non-participating groups, and the out-of-range error contract

Versions/languages covered, as requested in the function PR review:

  • docs/.../regexp-extract-all.md (dev, en)
  • i18n/zh-CN/docusaurus-plugin-content-docs/current/.../regexp-extract-all.md (dev, zh)
  • versioned_docs/version-4.x/.../regexp-extract-all.md (4.x, en)
  • i18n/zh-CN/docusaurus-plugin-content-docs/version-4.x/.../regexp-extract-all.md (4.x, zh)

Related

Function implementation: apache/doris#66050

Document the third parameter (idx) of REGEXP_EXTRACT_ALL in both the
dev and 4.x versions, in English and Chinese: group selection
semantics (0 = whole match, 1 = default first group), empty strings
for non-participating groups, and the out-of-range error contract.

Function change: apache/doris#66050
@ccl125

ccl125 commented Oct 10, 2026

Copy link
Copy Markdown
Author

Hi — gentle ping. This PR documents the optional group-index argument of regexp_extract_all, as the docs companion to apache/doris#66050. It has been open for ~2.5 months without review. Could a committer take a look? Happy to adjust anything. cc @linrrzqqq — thanks!

@morningman
morningman marked this pull request as draft October 16, 2026 21:26
@ccl125

ccl125 commented Oct 20, 2026

Copy link
Copy Markdown
Author

Thanks @morningman! Just to confirm the expected order: this docs PR should stay in draft until apache/doris#66050 is merged, right? I'll keep an eye on the code PR and mark this ready for review once it lands.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant