You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@kylin.apache.org by GitBox <gi...@apache.org> on 2020/02/12 03:08:22 UTC

[GitHub] [kylin] zhoukangcn opened a new pull request #1105: KYLIN-4377: allow project administrator to operate hybrid model

zhoukangcn opened a new pull request #1105: KYLIN-4377: allow project administrator to operate hybrid model
URL: https://github.com/apache/kylin/pull/1105
 
 
   see: https://issues.apache.org/jira/browse/KYLIN-4377

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [kylin] nichunen merged pull request #1105: KYLIN-4377: allow project administrator to operate hybrid model

Posted by GitBox <gi...@apache.org>.
nichunen merged pull request #1105: KYLIN-4377: allow project administrator to operate hybrid model
URL: https://github.com/apache/kylin/pull/1105
 
 
   

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [kylin] coveralls commented on issue #1105: KYLIN-4377: allow project administrator to operate hybrid model

Posted by GitBox <gi...@apache.org>.
coveralls commented on issue #1105: KYLIN-4377: allow project administrator to operate hybrid model
URL: https://github.com/apache/kylin/pull/1105#issuecomment-585013167
 
 
   ## Pull Request Test Coverage Report for [Build 5612](https://coveralls.io/builds/28674420)
   
   * **0** of **0**   changed or added relevant lines in **0** files are covered.
   * **10** unchanged lines in **1** file lost coverage.
   * Overall coverage decreased (**-0.005%**) to **27.517%**
   
   ---
   
   
   |  Files with Coverage Reduction | New Missed Lines | % |
   | :-----|--------------|--: |
   | [core-cube/src/main/java/org/apache/kylin/cube/inmemcubing/MemDiskStore.java](https://coveralls.io/builds/28674420/source?filename=core-cube%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcube%2Finmemcubing%2FMemDiskStore.java#L193) | 10 | 75.99% |
   <!-- | **Total:** | **10** |  | -->
   
   |  Totals | [![Coverage Status](https://coveralls.io/builds/28674420/badge)](https://coveralls.io/builds/28674420) |
   | :-- | --: |
   | Change from base [Build 5602](https://coveralls.io/builds/28603095): |  -0.005% |
   | Covered Lines: | 24183 |
   | Relevant Lines: | 87883 |
   
   ---
   ##### 💛  - [Coveralls](https://coveralls.io)
   

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [kylin] codecov-io commented on issue #1105: KYLIN-4377: allow project administrator to operate hybrid model

Posted by GitBox <gi...@apache.org>.
codecov-io commented on issue #1105: KYLIN-4377: allow project administrator to operate hybrid model
URL: https://github.com/apache/kylin/pull/1105#issuecomment-585011210
 
 
   # [Codecov](https://codecov.io/gh/apache/kylin/pull/1105?src=pr&el=h1) Report
   > Merging [#1105](https://codecov.io/gh/apache/kylin/pull/1105?src=pr&el=desc) into [master](https://codecov.io/gh/apache/kylin/commit/4e5b9a31ba88292ef6e56129494e034307fe4f01?src=pr&el=desc) will **decrease** coverage by `0.02%`.
   > The diff coverage is `n/a`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/kylin/pull/1105/graphs/tree.svg?width=650&token=JawVgbgsVo&height=150&src=pr)](https://codecov.io/gh/apache/kylin/pull/1105?src=pr&el=tree)
   
   ```diff
   @@             Coverage Diff              @@
   ##             master    #1105      +/-   ##
   ============================================
   - Coverage     25.07%   25.05%   -0.03%     
   + Complexity     6226     6224       -2     
   ============================================
     Files          1440     1440              
     Lines         87883    87883              
     Branches      12291    12291              
   ============================================
   - Hits          22039    22015      -24     
   - Misses        63688    63703      +15     
   - Partials       2156     2165       +9
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/kylin/pull/1105?src=pr&el=tree) | Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | [...he/kylin/dict/lookup/cache/RocksDBLookupTable.java](https://codecov.io/gh/apache/kylin/pull/1105/diff?src=pr&el=tree#diff-Y29yZS1kaWN0aW9uYXJ5L3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9reWxpbi9kaWN0L2xvb2t1cC9jYWNoZS9Sb2Nrc0RCTG9va3VwVGFibGUuamF2YQ==) | `72.97% <0%> (-5.41%)` | `6% <0%> (-1%)` | |
   | [...rg/apache/kylin/cube/inmemcubing/MemDiskStore.java](https://codecov.io/gh/apache/kylin/pull/1105/diff?src=pr&el=tree#diff-Y29yZS1jdWJlL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9reWxpbi9jdWJlL2lubWVtY3ViaW5nL01lbURpc2tTdG9yZS5qYXZh) | `65.95% <0%> (-5.17%)` | `7% <0%> (ø)` | |
   | [...org/apache/kylin/rest/util/QueryRequestLimits.java](https://codecov.io/gh/apache/kylin/pull/1105/diff?src=pr&el=tree#diff-c2VydmVyLWJhc2Uvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL3Jlc3QvdXRpbC9RdWVyeVJlcXVlc3RMaW1pdHMuamF2YQ==) | `35.71% <0%> (-4.77%)` | `5% <0%> (-1%)` | |
   | [...ream/core/storage/columnar/ColumnarStoreCache.java](https://codecov.io/gh/apache/kylin/pull/1105/diff?src=pr&el=tree#diff-c3RyZWFtLWNvcmUvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL3N0cmVhbS9jb3JlL3N0b3JhZ2UvY29sdW1uYXIvQ29sdW1uYXJTdG9yZUNhY2hlLmphdmE=) | `52.27% <0%> (-3.41%)` | `8% <0%> (ø)` | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/kylin/pull/1105?src=pr&el=continue).
   > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by [Codecov](https://codecov.io/gh/apache/kylin/pull/1105?src=pr&el=footer). Last update [4e5b9a3...94aa4ef](https://codecov.io/gh/apache/kylin/pull/1105?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services