-
Notifications
You must be signed in to change notification settings - Fork 597
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
feat(expr): add regexp_split_to_array
#12844
Conversation
Codecov Report
@@ Coverage Diff @@
## main #12844 +/- ##
==========================================
- Coverage 69.18% 69.17% -0.02%
==========================================
Files 1489 1489
Lines 245832 245901 +69
==========================================
+ Hits 170083 170091 +8
- Misses 75749 75810 +61
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 2 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Can you enable regress tests in risingwave/src/tests/regress/data/sql/strings.sql Lines 235 to 261 in e24370d
--@ here (See https://github.com/risingwavelabs/risingwave/blob/f2a3fd021059e680b35b24c63cff5f8dbe9f9d5f/src/tests/regress/README.md for details)
You can only enable tests that passed now. Not necessary to support them all. |
🤔
|
Signed-off-by: TennyZhuang <[email protected]>
This is due to, currently, we treat the delimiter which contains empty separated field as pure empty, but postgres treats these as empty string. |
I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
resolve #12509.
Checklist
./risedev check
(or alias,./risedev c
)Documentation
Release note
Support
regexp_split_to_array
function