Skip to content

Conversation

@walterddr
Copy link
Contributor

@walterddr walterddr commented Sep 16, 2022

by running the hard-coded queries list first

  • make relRoot info available in query plan
  • uses fields / collation / hints properly before returning results
  • fix SQL conformance
  • some other bug fixes

this PR also listed out TODOs for features we don't support right now.

Rong Rong added 4 commits September 15, 2022 19:53
- masked out columns should not be returned.
- make API more meaningful
- only tests with standard SQL
- ignore non-standard SQL
- add todo for tests that doesn't work
@codecov-commenter
Copy link

codecov-commenter commented Sep 16, 2022

Codecov Report

Merging #9412 (0bda3c7) into master (bfa2a5a) will increase coverage by 0.04%.
The diff coverage is 95.00%.

@@             Coverage Diff              @@
##             master    #9412      +/-   ##
============================================
+ Coverage     69.74%   69.79%   +0.04%     
- Complexity     5090     5093       +3     
============================================
  Files          1890     1890              
  Lines        100683   100756      +73     
  Branches      15337    15350      +13     
============================================
+ Hits          70220    70320     +100     
+ Misses        25475    25441      -34     
- Partials       4988     4995       +7     
Flag Coverage Δ
integration1 26.08% <2.50%> (+0.05%) ⬆️
integration2 24.70% <0.00%> (-0.05%) ⬇️
unittests1 66.99% <94.87%> (-0.03%) ⬇️
unittests2 15.37% <92.50%> (+<0.01%) ⬆️

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

Impacted Files Coverage Δ
...rg/apache/pinot/query/service/QueryDispatcher.java 77.41% <92.59%> (+7.27%) ⬆️
...requesthandler/MultiStageBrokerRequestHandler.java 59.15% <100.00%> (-7.90%) ⬇️
.../java/org/apache/pinot/query/QueryEnvironment.java 83.87% <100.00%> (+0.26%) ⬆️
...java/org/apache/pinot/query/planner/QueryPlan.java 100.00% <100.00%> (ø)
...ache/pinot/query/planner/logical/StagePlanner.java 93.91% <100.00%> (+0.05%) ⬆️
...ava/org/apache/pinot/query/validate/Validator.java 100.00% <100.00%> (ø)
...nt/local/startree/v2/store/StarTreeDataSource.java 40.00% <0.00%> (-13.34%) ⬇️
...ache/pinot/core/operator/docidsets/OrDocIdSet.java 86.36% <0.00%> (-11.37%) ⬇️
...ot/segment/local/startree/OffHeapStarTreeNode.java 65.90% <0.00%> (-6.82%) ⬇️
...r/validation/RealtimeSegmentValidationManager.java 71.62% <0.00%> (-5.41%) ⬇️
... and 35 more

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

+ "(Diff >= 300 AND Diff < 500) OR Diff < -500 ORDER BY Diff DESC";
testQuery(query);

// IN_ID_SET
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Jackie-Jiang @61yao the following are turned of with multistage for now.

@walterddr walterddr merged commit 2d6665b into apache:master Sep 16, 2022
61yao pushed a commit to 61yao/pinot that referenced this pull request Oct 3, 2022
by running the hard-coded queries list first

- make relRoot info available in query plan
- uses fields / collation / hints properly before returning results
- fix SQL conformance
- some other bug fixes

this PR also listed out TODOs for features we don't support right now.

Co-authored-by: Rong Rong <rongr@startree.ai>
@walterddr walterddr deleted the pr_hook_h2_itcase branch December 6, 2023 16:22
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.

3 participants