You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by "codecov-commenter (via GitHub)" <gi...@apache.org> on 2023/03/01 00:39:14 UTC

[GitHub] [pinot] codecov-commenter commented on pull request #10357: Enhance select order-by combine to use merge sort

codecov-commenter commented on PR #10357:
URL: https://github.com/apache/pinot/pull/10357#issuecomment-1449137938

   # [Codecov](https://codecov.io/gh/apache/pinot/pull/10357?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) Report
   > Merging [#10357](https://codecov.io/gh/apache/pinot/pull/10357?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (7281ad4) into [master](https://codecov.io/gh/apache/pinot/commit/246f4db352b282d576be44348b5652638894e8e4?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (246f4db) will **decrease** coverage by `7.58%`.
   > The diff coverage is `93.26%`.
   
   ```diff
   @@             Coverage Diff              @@
   ##             master   #10357      +/-   ##
   ============================================
   - Coverage     32.06%   24.48%   -7.58%     
   + Complexity      245       49     -196     
   ============================================
     Files          2035     2023      -12     
     Lines        110228   109893     -335     
     Branches      16748    16716      -32     
   ============================================
   - Hits          35342    26905    -8437     
   - Misses        71742    80186    +8444     
   + Partials       3144     2802     -342     
   ```
   
   | Flag | Coverage Δ | |
   |---|---|---|
   | integration1 | `24.48% <93.26%> (+0.02%)` | :arrow_up: |
   | unittests2 | `?` | |
   
   Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#carryforward-flags-in-the-pull-request-comment) to find out more.
   
   | [Impacted Files](https://codecov.io/gh/apache/pinot/pull/10357?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) | Coverage Δ | |
   |---|---|---|
   | [...erator/blocks/results/AggregationResultsBlock.java](https://codecov.io/gh/apache/pinot/pull/10357?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-cGlub3QtY29yZS9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvcGlub3QvY29yZS9vcGVyYXRvci9ibG9ja3MvcmVzdWx0cy9BZ2dyZWdhdGlvblJlc3VsdHNCbG9jay5qYXZh) | `44.44% <ø> (ø)` | |
   | [...core/operator/blocks/results/BaseResultsBlock.java](https://codecov.io/gh/apache/pinot/pull/10357?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-cGlub3QtY29yZS9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvcGlub3QvY29yZS9vcGVyYXRvci9ibG9ja3MvcmVzdWx0cy9CYXNlUmVzdWx0c0Jsb2NrLmphdmE=) | `71.69% <ø> (ø)` | |
   | [.../operator/blocks/results/DistinctResultsBlock.java](https://codecov.io/gh/apache/pinot/pull/10357?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-cGlub3QtY29yZS9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvcGlub3QvY29yZS9vcGVyYXRvci9ibG9ja3MvcmVzdWx0cy9EaXN0aW5jdFJlc3VsdHNCbG9jay5qYXZh) | `100.00% <ø> (ø)` | |
   | [...operator/blocks/results/ExceptionResultsBlock.java](https://codecov.io/gh/apache/pinot/pull/10357?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-cGlub3QtY29yZS9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvcGlub3QvY29yZS9vcGVyYXRvci9ibG9ja3MvcmVzdWx0cy9FeGNlcHRpb25SZXN1bHRzQmxvY2suamF2YQ==) | `66.66% <ø> (ø)` | |
   | [...e/operator/blocks/results/ExplainResultsBlock.java](https://codecov.io/gh/apache/pinot/pull/10357?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-cGlub3QtY29yZS9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvcGlub3QvY29yZS9vcGVyYXRvci9ibG9ja3MvcmVzdWx0cy9FeHBsYWluUmVzdWx0c0Jsb2NrLmphdmE=) | `70.37% <ø> (ø)` | |
   | [...e/operator/blocks/results/GroupByResultsBlock.java](https://codecov.io/gh/apache/pinot/pull/10357?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-cGlub3QtY29yZS9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvcGlub3QvY29yZS9vcGVyYXRvci9ibG9ja3MvcmVzdWx0cy9Hcm91cEJ5UmVzdWx0c0Jsb2NrLmphdmE=) | `55.46% <ø> (+0.84%)` | :arrow_up: |
   | [.../operator/blocks/results/MetadataResultsBlock.java](https://codecov.io/gh/apache/pinot/pull/10357?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-cGlub3QtY29yZS9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvcGlub3QvY29yZS9vcGVyYXRvci9ibG9ja3MvcmVzdWx0cy9NZXRhZGF0YVJlc3VsdHNCbG9jay5qYXZh) | `0.00% <ø> (ø)` | |
   | [...re/operator/combine/merger/ResultsBlockMerger.java](https://codecov.io/gh/apache/pinot/pull/10357?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-cGlub3QtY29yZS9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvcGlub3QvY29yZS9vcGVyYXRvci9jb21iaW5lL21lcmdlci9SZXN1bHRzQmxvY2tNZXJnZXIuamF2YQ==) | `100.00% <ø> (ø)` | |
   | [...core/query/selection/SelectionOperatorService.java](https://codecov.io/gh/apache/pinot/pull/10357?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-cGlub3QtY29yZS9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvcGlub3QvY29yZS9xdWVyeS9zZWxlY3Rpb24vU2VsZWN0aW9uT3BlcmF0b3JTZXJ2aWNlLmphdmE=) | `64.58% <ø> (ø)` | |
   | [...operator/query/LinearSelectionOrderByOperator.java](https://codecov.io/gh/apache/pinot/pull/10357?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-cGlub3QtY29yZS9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvcGlub3QvY29yZS9vcGVyYXRvci9xdWVyeS9MaW5lYXJTZWxlY3Rpb25PcmRlckJ5T3BlcmF0b3IuamF2YQ==) | `72.22% <87.09%> (+5.29%)` | :arrow_up: |
   | ... and [324 more](https://codecov.io/gh/apache/pinot/pull/10357?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) | |
   
   :mega: We’re building smart automated test selection to slash your CI/CD build times. [Learn more](https://about.codecov.io/iterative-testing/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org