Skip to content

[multistage] support like and regex like#9654

Merged
walterddr merged 2 commits intoapache:masterfrom
walterddr:pr_like
Oct 25, 2022
Merged

[multistage] support like and regex like#9654
walterddr merged 2 commits intoapache:masterfrom
walterddr:pr_like

Conversation

@walterddr
Copy link
Contributor

Support like and regex like

This prompts another v1 legacy issue, not all function evaluation has predicate/transform/scalar version - needs a clean up to make sure any function that can be evaluated at predicate time / transform time, should ALWAYS be usable at reduce/multistage.

@walterddr walterddr marked this pull request as ready for review October 25, 2022 18:04
@walterddr walterddr added the multi-stage Related to the multi-stage query engine label Oct 25, 2022
@codecov-commenter
Copy link

Codecov Report

Merging #9654 (8dc5f59) into master (3de1c54) will decrease coverage by 40.41%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             master    #9654       +/-   ##
=============================================
- Coverage     68.63%   28.21%   -40.42%     
+ Complexity     4920       53     -4867     
=============================================
  Files          1947     1934       -13     
  Lines        104168   103808      -360     
  Branches      15796    15759       -37     
=============================================
- Hits          71496    29290    -42206     
- Misses        27556    71650    +44094     
+ Partials       5116     2868     -2248     
Flag Coverage Δ
integration1 25.94% <0.00%> (+0.07%) ⬆️
integration2 24.74% <0.00%> (?)
unittests1 ?
unittests2 ?

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

Impacted Files Coverage Δ
.../pinot/common/function/scalar/StringFunctions.java 34.82% <0.00%> (-38.33%) ⬇️
...query/runtime/operator/operands/FilterOperand.java 0.00% <0.00%> (-65.39%) ⬇️
...in/java/org/apache/pinot/spi/utils/BytesUtils.java 0.00% <0.00%> (-100.00%) ⬇️
...java/org/apache/pinot/spi/trace/BaseRecording.java 0.00% <0.00%> (-100.00%) ⬇️
...java/org/apache/pinot/spi/trace/NoOpRecording.java 0.00% <0.00%> (-100.00%) ⬇️
...ava/org/apache/pinot/spi/config/table/FSTType.java 0.00% <0.00%> (-100.00%) ⬇️
...ava/org/apache/pinot/spi/config/user/RoleType.java 0.00% <0.00%> (-100.00%) ⬇️
...ava/org/apache/pinot/spi/data/MetricFieldSpec.java 0.00% <0.00%> (-100.00%) ⬇️
...ava/org/apache/pinot/spi/stream/StreamMessage.java 0.00% <0.00%> (-100.00%) ⬇️
...java/org/apache/pinot/common/tier/TierFactory.java 0.00% <0.00%> (-100.00%) ⬇️
... and 1391 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@walterddr walterddr merged commit 62f6fe1 into apache:master Oct 25, 2022
@walterddr walterddr deleted the pr_like branch December 6, 2023 16:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

multi-stage Related to the multi-stage query engine

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants