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 2019/12/26 03:02:48 UTC

[GitHub] [kylin] codecov-io edited a comment on issue #971: KYLIN-3237 Not to allow segment deletion by REST API when its cube is…

codecov-io edited a comment on issue #971: KYLIN-3237 Not to allow segment deletion by REST API when its cube is…
URL: https://github.com/apache/kylin/pull/971#issuecomment-561054570
 
 
   # [Codecov](https://codecov.io/gh/apache/kylin/pull/971?src=pr&el=h1) Report
   > Merging [#971](https://codecov.io/gh/apache/kylin/pull/971?src=pr&el=desc) into [master](https://codecov.io/gh/apache/kylin/commit/8a8d1b0a081b0040284bb612254bce5007cf0729?src=pr&el=desc) will **decrease** coverage by `0.36%`.
   > The diff coverage is `0%`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/kylin/pull/971/graphs/tree.svg?width=650&token=JawVgbgsVo&height=150&src=pr)](https://codecov.io/gh/apache/kylin/pull/971?src=pr&el=tree)
   
   ```diff
   @@             Coverage Diff              @@
   ##             master     #971      +/-   ##
   ============================================
   - Coverage     25.48%   25.11%   -0.37%     
   - Complexity     6148     6179      +31     
   ============================================
     Files          1413     1433      +20     
     Lines         85495    87260    +1765     
     Branches      11975    12215     +240     
   ============================================
   + Hits          21790    21918     +128     
   - Misses        61584    63196    +1612     
   - Partials       2121     2146      +25
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/kylin/pull/971?src=pr&el=tree) | Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | [...c/main/java/org/apache/kylin/rest/msg/Message.java](https://codecov.io/gh/apache/kylin/pull/971/diff?src=pr&el=tree#diff-c2VydmVyLWJhc2Uvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL3Jlc3QvbXNnL01lc3NhZ2UuamF2YQ==) | `0% <0%> (ø)` | `0 <0> (ø)` | :arrow_down: |
   | [...main/java/org/apache/kylin/rest/msg/CnMessage.java](https://codecov.io/gh/apache/kylin/pull/971/diff?src=pr&el=tree#diff-c2VydmVyLWJhc2Uvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL3Jlc3QvbXNnL0NuTWVzc2FnZS5qYXZh) | `0% <0%> (ø)` | `0 <0> (ø)` | :arrow_down: |
   | [...ava/org/apache/kylin/rest/service/CubeService.java](https://codecov.io/gh/apache/kylin/pull/971/diff?src=pr&el=tree#diff-c2VydmVyLWJhc2Uvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL3Jlc3Qvc2VydmljZS9DdWJlU2VydmljZS5qYXZh) | `0% <0%> (ø)` | `0 <0> (ø)` | :arrow_down: |
   | [...ylin/source/jdbc/metadata/JdbcMetadataFactory.java](https://codecov.io/gh/apache/kylin/pull/971/diff?src=pr&el=tree#diff-c291cmNlLWpkYmMvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL3NvdXJjZS9qZGJjL21ldGFkYXRhL0pkYmNNZXRhZGF0YUZhY3RvcnkuamF2YQ==) | `0% <0%> (-25%)` | `0% <0%> (-1%)` | |
   | [...ain/java/org/apache/kylin/source/jdbc/SqlUtil.java](https://codecov.io/gh/apache/kylin/pull/971/diff?src=pr&el=tree#diff-c291cmNlLWpkYmMvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL3NvdXJjZS9qZGJjL1NxbFV0aWwuamF2YQ==) | `43.24% <0%> (-13.52%)` | `20% <0%> (-2%)` | |
   | [...ylin/source/jdbc/extensible/JdbcHiveInputBase.java](https://codecov.io/gh/apache/kylin/pull/971/diff?src=pr&el=tree#diff-c291cmNlLWpkYmMvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL3NvdXJjZS9qZGJjL2V4dGVuc2libGUvSmRiY0hpdmVJbnB1dEJhc2UuamF2YQ==) | `71.42% <0%> (-12.58%)` | `2% <0%> (ø)` | |
   | [...rg/apache/kylin/source/jdbc/JdbcHiveInputBase.java](https://codecov.io/gh/apache/kylin/pull/971/diff?src=pr&el=tree#diff-c291cmNlLWpkYmMvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL3NvdXJjZS9qZGJjL0pkYmNIaXZlSW5wdXRCYXNlLmphdmE=) | `9.02% <0%> (-12.02%)` | `7% <0%> (-1%)` | |
   | [...he/kylin/engine/mr/common/MapReduceExecutable.java](https://codecov.io/gh/apache/kylin/pull/971/diff?src=pr&el=tree#diff-ZW5naW5lLW1yL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9reWxpbi9lbmdpbmUvbXIvY29tbW9uL01hcFJlZHVjZUV4ZWN1dGFibGUuamF2YQ==) | `13.33% <0%> (-9.4%)` | `6% <0%> (ø)` | |
   | [.../apache/kylin/dimension/TimeDerivedColumnType.java](https://codecov.io/gh/apache/kylin/pull/971/diff?src=pr&el=tree#diff-Y29yZS1tZXRhZGF0YS9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUva3lsaW4vZGltZW5zaW9uL1RpbWVEZXJpdmVkQ29sdW1uVHlwZS5qYXZh) | `79.16% <0%> (-8.34%)` | `9% <0%> (ø)` | |
   | [...ylin/job/impl/threadpool/DefaultFetcherRunner.java](https://codecov.io/gh/apache/kylin/pull/971/diff?src=pr&el=tree#diff-Y29yZS1qb2Ivc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL2pvYi9pbXBsL3RocmVhZHBvb2wvRGVmYXVsdEZldGNoZXJSdW5uZXIuamF2YQ==) | `54.38% <0%> (-8.12%)` | `6% <0%> (ø)` | |
   | ... and [97 more](https://codecov.io/gh/apache/kylin/pull/971/diff?src=pr&el=tree-more) | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/kylin/pull/971?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/971?src=pr&el=footer). Last update [8a8d1b0...36ef71e](https://codecov.io/gh/apache/kylin/pull/971?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