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/27 06:58:50 UTC
[GitHub] [kylin] codecov-io edited a comment on issue #987: KYLIN-4294: Add
http api for metrics
codecov-io edited a comment on issue #987: KYLIN-4294: Add http api for metrics
URL: https://github.com/apache/kylin/pull/987#issuecomment-564359873
# [Codecov](https://codecov.io/gh/apache/kylin/pull/987?src=pr&el=h1) Report
> Merging [#987](https://codecov.io/gh/apache/kylin/pull/987?src=pr&el=desc) into [master](https://codecov.io/gh/apache/kylin/commit/727de71312270485a342d7a02e13658dc9d9a774?src=pr&el=desc) will **decrease** coverage by `0.33%`.
> The diff coverage is `0%`.
[](https://codecov.io/gh/apache/kylin/pull/987?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #987 +/- ##
===========================================
- Coverage 25.43% 25.1% -0.34%
- Complexity 6156 6179 +23
===========================================
Files 1415 1435 +20
Lines 85877 87299 +1422
Branches 12042 12220 +178
===========================================
+ Hits 21845 21918 +73
- Misses 61905 63234 +1329
- Partials 2127 2147 +20
```
| [Impacted Files](https://codecov.io/gh/apache/kylin/pull/987?src=pr&el=tree) | Coverage Δ | Complexity Δ | |
|---|---|---|---|
| [...pache/kylin/rest/controller/MetricsController.java](https://codecov.io/gh/apache/kylin/pull/987/diff?src=pr&el=tree#diff-c2VydmVyLWJhc2Uvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL3Jlc3QvY29udHJvbGxlci9NZXRyaWNzQ29udHJvbGxlci5qYXZh) | `0% <0%> (ø)` | `0 <0> (?)` | |
| [.../org/apache/kylin/rest/service/MetricsService.java](https://codecov.io/gh/apache/kylin/pull/987/diff?src=pr&el=tree#diff-c2VydmVyLWJhc2Uvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL3Jlc3Qvc2VydmljZS9NZXRyaWNzU2VydmljZS5qYXZh) | `0% <0%> (ø)` | `0 <0> (?)` | |
| [...ylin/source/jdbc/metadata/JdbcMetadataFactory.java](https://codecov.io/gh/apache/kylin/pull/987/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/987/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/987/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/987/diff?src=pr&el=tree#diff-c291cmNlLWpkYmMvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL3NvdXJjZS9qZGJjL0pkYmNIaXZlSW5wdXRCYXNlLmphdmE=) | `9.05% <0%> (-11.98%)` | `7% <0%> (-1%)` | |
| [...ylin/job/impl/threadpool/DefaultFetcherRunner.java](https://codecov.io/gh/apache/kylin/pull/987/diff?src=pr&el=tree#diff-Y29yZS1qb2Ivc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL2pvYi9pbXBsL3RocmVhZHBvb2wvRGVmYXVsdEZldGNoZXJSdW5uZXIuamF2YQ==) | `54.38% <0%> (-8.12%)` | `6% <0%> (ø)` | |
| [...he/kylin/dict/lookup/cache/RocksDBLookupTable.java](https://codecov.io/gh/apache/kylin/pull/987/diff?src=pr&el=tree#diff-Y29yZS1kaWN0aW9uYXJ5L3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9reWxpbi9kaWN0L2xvb2t1cC9jYWNoZS9Sb2Nrc0RCTG9va3VwVGFibGUuamF2YQ==) | `72.97% <0%> (-5.41%)` | `6% <0%> (-1%)` | |
| [...org/apache/kylin/rest/util/QueryRequestLimits.java](https://codecov.io/gh/apache/kylin/pull/987/diff?src=pr&el=tree#diff-c2VydmVyLWJhc2Uvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL3Jlc3QvdXRpbC9RdWVyeVJlcXVlc3RMaW1pdHMuamF2YQ==) | `35.71% <0%> (-4.77%)` | `5% <0%> (-1%)` | |
| [...ylin/sdk/datasource/framework/conv/ConvMaster.java](https://codecov.io/gh/apache/kylin/pull/987/diff?src=pr&el=tree#diff-ZGF0YXNvdXJjZS1zZGsvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL3Nkay9kYXRhc291cmNlL2ZyYW1ld29yay9jb252L0NvbnZNYXN0ZXIuamF2YQ==) | `65.67% <0%> (-2.99%)` | `11% <0%> (ø)` | |
| ... and [77 more](https://codecov.io/gh/apache/kylin/pull/987/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/kylin/pull/987?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/987?src=pr&el=footer). Last update [727de71...3e988ef](https://codecov.io/gh/apache/kylin/pull/987?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