[ 
https://issues.apache.org/jira/browse/KYLIN-4794?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18071014#comment-18071014
 ] 

ASF GitHub Bot commented on KYLIN-4794:
---------------------------------------

codecov-commenter commented on PR #1614:
URL: https://github.com/apache/kylin/pull/1614#issuecomment-4186939602

   ## 
[Codecov](https://app.codecov.io/gh/apache/kylin/pull/1614?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 Report
   :x: Patch coverage is `0%` with `27 lines` in your changes missing coverage. 
Please review.
   :white_check_mark: Project coverage is 25.39%. Comparing base 
([`0250081`](https://app.codecov.io/gh/apache/kylin/commit/02500818d85955b1125340eaa19ab88a634a5181?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache))
 to head 
([`89d5652`](https://app.codecov.io/gh/apache/kylin/commit/89d56528b35e3b8d5eabe01b2d8264fe753891fe?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
   
   | [Files with missing 
lines](https://app.codecov.io/gh/apache/kylin/pull/1614?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 | Patch % | Lines |
   |---|---|---|
   | 
[...va/org/apache/kylin/query/routing/QueryRouter.java](https://app.codecov.io/gh/apache/kylin/pull/1614?src=pr&el=tree&filepath=query%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fquery%2Frouting%2FQueryRouter.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-cXVlcnkvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL3F1ZXJ5L3JvdXRpbmcvUXVlcnlSb3V0ZXIuamF2YQ==)
 | 0.00% | [15 Missing :warning: 
](https://app.codecov.io/gh/apache/kylin/pull/1614?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 |
   | 
[.../kylin/query/enumerator/LookupTableEnumerator.java](https://app.codecov.io/gh/apache/kylin/pull/1614?src=pr&el=tree&filepath=query%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fquery%2Fenumerator%2FLookupTableEnumerator.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-cXVlcnkvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL3F1ZXJ5L2VudW1lcmF0b3IvTG9va3VwVGFibGVFbnVtZXJhdG9yLmphdmE=)
 | 0.00% | [12 Missing :warning: 
](https://app.codecov.io/gh/apache/kylin/pull/1614?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 |
   
   <details><summary>Additional details and impacted files</summary>
   
   
   
   ```diff
   @@             Coverage Diff              @@
   ##             master    #1614      +/-   ##
   ============================================
   - Coverage     25.42%   25.39%   -0.03%     
   + Complexity     6765     6761       -4     
   ============================================
     Files          1508     1508              
     Lines         93918    93986      +68     
     Branches      13158    13172      +14     
   ============================================
   - Hits          23877    23872       -5     
   - Misses        67662    67734      +72     
   - Partials       2379     2380       +1     
   ```
   </details>
   
   [:umbrella: View full report in Codecov by 
Sentry](https://app.codecov.io/gh/apache/kylin/pull/1614?dropdown=coverage&src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
   
   :loudspeaker: Have feedback on the report? [Share it 
here](https://about.codecov.io/codecov-pr-comment-feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
   <details><summary> :rocket: New features to boost your workflow: </summary>
   
   - :snowflake: [Test 
Analytics](https://docs.codecov.com/docs/test-analytics): Detect flaky tests, 
report on failures, and find test suite problems.
   - :package: [JS Bundle 
Analysis](https://docs.codecov.com/docs/javascript-bundle-analysis): Save 
yourself from yourself by tracking and limiting bundle sizes in JS merges.
   </details>




> Make it possible to force hit a cube set for sqls with cube join
> ----------------------------------------------------------------
>
>                 Key: KYLIN-4794
>                 URL: https://issues.apache.org/jira/browse/KYLIN-4794
>             Project: Kylin
>          Issue Type: New Feature
>            Reporter: JiangYang
>            Assignee: JiangYang
>            Priority: Major
>             Fix For: v3.1.2
>
>
> Currently only single cube can be specified. However, when with complex sqls 
> with multiple subqueries for cube joins, multiple cubes will be hit. In this 
> case, we need to specify a cube set rather than a single cube.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to