Add support for regexpReplace scalar function

[vvivekiyer]

Fixes issue = #9079
Label = feature

This PR adds a scalar function to perform REGEXP REPLACE in Pinot. The semantics of using this scalar function is as follows:

• If matches is not found, inputStr is returned as such.
• By default, all occurrences of the matched patterns are replaced unless the occurence parameter is not -1.
• Default match flag is case sensitive. Can be overriden with the flag parameter

Parameters:

1. inputStr (required): Input string against which regexp match and replace should be performed.
2. matchStr (required): String or regexp to match against inputStr
3. replaceStr (required): String or regexp to replace if a pattern is found.
4. startPos (optional): Starting index from where matching should be performed. Default value is 0.
5. occurrence (optional): The occurrence of the matched pattern must be replaced. Default is -1 where all found patterns will be replaced.
6. flag (optional): Single character flag that controls how the regex finds matches in inputStr. Default flag of empty is case sensitive match. If an incorrect flag is specified, the function applies default case sensitive match. Only one flag can be specified. Supported flags are:

• i -> Case insensitive
• m -> Treats the string to match as multiple lines
• x -> Add comments to the regular expression

We can support more flags in the future.

[codecov-commenter]

Codecov Report

Merging #9123 (655b1b4) into master (913f0d8) will increase coverage by 2.28%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master    #9123      +/-   ##
============================================
+ Coverage     61.23%   63.51%   +2.28%     
+ Complexity     4842     4735     -107     
============================================
  Files          1836     1799      -37     
  Lines         97524    95732    -1792     
  Branches      14703    14497     -206     
============================================
+ Hits          59716    60806    +1090     
+ Misses        33342    30542    -2800     
+ Partials       4466     4384      -82     

Flag	Coverage Δ
integration1	?
unittests1	66.97% <0.00%> (+0.01%)	⬆️
unittests2	15.37% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
.../pinot/common/function/scalar/StringFunctions.java	54.90% <0.00%> (-17.25%)	⬇️
...va/org/apache/pinot/core/routing/RoutingTable.java	0.00% <0.00%> (-100.00%)	⬇️
...va/org/apache/pinot/common/config/NettyConfig.java	0.00% <0.00%> (-100.00%)	⬇️
...a/org/apache/pinot/common/metrics/MinionMeter.java	0.00% <0.00%> (-100.00%)	⬇️
...g/apache/pinot/common/metrics/ControllerMeter.java	0.00% <0.00%> (-100.00%)	⬇️
.../apache/pinot/common/metrics/BrokerQueryPhase.java	0.00% <0.00%> (-100.00%)	⬇️
.../apache/pinot/common/metrics/MinionQueryPhase.java	0.00% <0.00%> (-100.00%)	⬇️
...ache/pinot/server/access/AccessControlFactory.java	0.00% <0.00%> (-100.00%)	⬇️
...he/pinot/common/messages/TableDeletionMessage.java	0.00% <0.00%> (-100.00%)	⬇️
...pinot/core/data/manager/realtime/TimerService.java	0.00% <0.00%> (-100.00%)	⬇️
... and 557 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[vvivekiyer]

@siddharthteotia @Jackie-Jiang please take a look and provide feedback. Thanks!

[siddharthteotia]

@vvivekiyer - can you rebase this once and also look at test failures ?

[vvivekiyer]

@siddharthteotia Rebased and reran the tests. Please review.

[siddharthteotia]

(nit) startPos -> matchStartPos ?

[vvivekiyer]

Done.

[siddharthteotia]

Not sure I follow the purpose / use of this flag. Can you please elaborate ?

[siddharthteotia]

Suggest not handling m and x now. We can add them later and keep it simple for now

[vvivekiyer]

Removed m and x.

[siddharthteotia]

@vvivekiyer - can you please rebase on top of #9114

[vvivekiyer]

@siddharthteotia rebased and all tests are passing now.

[siddharthteotia]

@vvivekiyer - let's please add user docs to gitbook

[Jackie-Jiang]

Not specific to this PR, but suggest using draft to mark the WIP PR instead of putting it as the title.
I saw the WIP; Do not Review was removed 5 days ago, but the github notification emails are still grouped under the original name, which is very confusing. I thought this PR is not yet ready for review, but is already merged...

[vvivekiyer]

@Jackie-Jiang Sure, makes sense.
If there is any feedback from your review, please let me know. I can address them as a followup.

[vvivekiyer]

User docs merged - https://app.gitbook.com/o/-LtRX9NwSr7Ga7zA4piL/s/-LtH6nl58DdnZnelPdTc-887967055/~/changes/9igVuyW1iAOxUO2FbIVg/configuration-reference/functions/regexpreplace