[
https://issues.apache.org/jira/browse/KYLIN-4855?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17257963#comment-17257963
]
ASF GitHub Bot commented on KYLIN-4855:
---------------------------------------
codecov-io edited a comment on pull request #1534:
URL: https://github.com/apache/kylin/pull/1534#issuecomment-753489521
# [Codecov](https://codecov.io/gh/apache/kylin/pull/1534?src=pr&el=h1) Report
> Merging
[#1534](https://codecov.io/gh/apache/kylin/pull/1534?src=pr&el=desc) (2b0fa3e)
into
[master](https://codecov.io/gh/apache/kylin/commit/c962a233672245c92db8ea29ee2f2540adbcc708?el=desc)
(c962a23) will **decrease** coverage by `0.04%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/kylin/pull/1534?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #1534 +/- ##
============================================
- Coverage 25.46% 25.42% -0.05%
+ Complexity 6766 6764 -2
============================================
Files 1507 1508 +1
Lines 93795 93915 +120
Branches 13135 13157 +22
============================================
- Hits 23888 23874 -14
- Misses 67531 67661 +130
- Partials 2376 2380 +4
```
| [Impacted
Files](https://codecov.io/gh/apache/kylin/pull/1534?src=pr&el=tree) | Coverage
Δ | Complexity Δ | |
|---|---|---|---|
|
[.../apache/kylin/tool/query/ProbabilityGenerator.java](https://codecov.io/gh/apache/kylin/pull/1534/diff?src=pr&el=tree#diff-dG9vbC9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUva3lsaW4vdG9vbC9xdWVyeS9Qcm9iYWJpbGl0eUdlbmVyYXRvci5qYXZh)
| `71.05% <0.00%> (-5.27%)` | `11.00% <0.00%> (ø%)` | |
|
[...core/storage/columnar/GeneralColumnDataReader.java](https://codecov.io/gh/apache/kylin/pull/1534/diff?src=pr&el=tree#diff-c3RyZWFtLWNvcmUvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL3N0cmVhbS9jb3JlL3N0b3JhZ2UvY29sdW1uYXIvR2VuZXJhbENvbHVtbkRhdGFSZWFkZXIuamF2YQ==)
| `89.47% <0.00%> (-5.27%)` | `6.00% <0.00%> (-1.00%)` | |
|
[.../apache/kylin/cube/cuboid/TreeCuboidScheduler.java](https://codecov.io/gh/apache/kylin/pull/1534/diff?src=pr&el=tree#diff-Y29yZS1jdWJlL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9reWxpbi9jdWJlL2N1Ym9pZC9UcmVlQ3Vib2lkU2NoZWR1bGVyLmphdmE=)
| `63.84% <0.00%> (-2.31%)` | `0.00% <0.00%> (ø%)` | |
|
[...rg/apache/kylin/source/jdbc/JdbcHiveInputBase.java](https://codecov.io/gh/apache/kylin/pull/1534/diff?src=pr&el=tree#diff-c291cmNlLWpkYmMvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL3NvdXJjZS9qZGJjL0pkYmNIaXZlSW5wdXRCYXNlLmphdmE=)
| `7.92% <0.00%> (-1.14%)` | `7.00% <0.00%> (ø%)` | |
|
[...va/org/apache/kylin/tool/query/QueryGenerator.java](https://codecov.io/gh/apache/kylin/pull/1534/diff?src=pr&el=tree#diff-dG9vbC9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUva3lsaW4vdG9vbC9xdWVyeS9RdWVyeUdlbmVyYXRvci5qYXZh)
| `80.00% <0.00%> (-1.12%)` | `23.00% <0.00%> (-1.00%)` | |
|
[...rg/apache/kylin/cube/inmemcubing/MemDiskStore.java](https://codecov.io/gh/apache/kylin/pull/1534/diff?src=pr&el=tree#diff-Y29yZS1jdWJlL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9reWxpbi9jdWJlL2lubWVtY3ViaW5nL01lbURpc2tTdG9yZS5qYXZh)
| `70.21% <0.00%> (-0.92%)` | `7.00% <0.00%> (ø%)` | |
|
[.../java/org/apache/kylin/common/KylinConfigBase.java](https://codecov.io/gh/apache/kylin/pull/1534/diff?src=pr&el=tree#diff-Y29yZS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL2NvbW1vbi9LeWxpbkNvbmZpZ0Jhc2UuamF2YQ==)
| `12.46% <0.00%> (-0.05%)` | `51.00% <0.00%> (ø%)` | |
|
[.../org/apache/kylin/query/relnode/OLAPFilterRel.java](https://codecov.io/gh/apache/kylin/pull/1534/diff?src=pr&el=tree#diff-cXVlcnkvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL3F1ZXJ5L3JlbG5vZGUvT0xBUEZpbHRlclJlbC5qYXZh)
| `0.00% <0.00%> (ø)` | `0.00% <0.00%> (ø%)` | |
|
[...g/apache/kylin/query/relnode/OLAPAggregateRel.java](https://codecov.io/gh/apache/kylin/pull/1534/diff?src=pr&el=tree#diff-cXVlcnkvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL3F1ZXJ5L3JlbG5vZGUvT0xBUEFnZ3JlZ2F0ZVJlbC5qYXZh)
| `0.00% <0.00%> (ø)` | `0.00% <0.00%> (ø%)` | |
|
[...apache/kylin/storage/hbase/steps/HBaseMRSteps.java](https://codecov.io/gh/apache/kylin/pull/1534/diff?src=pr&el=tree#diff-c3RvcmFnZS1oYmFzZS9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUva3lsaW4vc3RvcmFnZS9oYmFzZS9zdGVwcy9IQmFzZU1SU3RlcHMuamF2YQ==)
| `0.00% <0.00%> (ø)` | `0.00% <0.00%> (ø%)` | |
| ... and [7
more](https://codecov.io/gh/apache/kylin/pull/1534/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at
Codecov](https://codecov.io/gh/apache/kylin/pull/1534?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/1534?src=pr&el=footer). Last
update
[c962a23...2b0fa3e](https://codecov.io/gh/apache/kylin/pull/1534?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:
[email protected]
> kylin metrics prefix bug in system-cube.sh
> ------------------------------------------
>
> Key: KYLIN-4855
> URL: https://issues.apache.org/jira/browse/KYLIN-4855
> Project: Kylin
> Issue Type: Bug
> Components: Environment
> Affects Versions: v2.6.0
> Environment: 2.6.0+
> Reporter: huajiewang
> Priority: Minor
> Labels: easyfix
> Fix For: Future
>
> Original Estimate: 1h
> Remaining Estimate: 1h
>
> After executed "system-cube.sh cron", the system cube will be added to the
> crontab, but the cube name is wrong, and it does not match the actual
> situation. After testing, it is found that the system-cube is
> There is a bug in the handling of kylin metrics prefix
> In the system-cube.sh script, the prefix is "KYLIN". The logic is wrong. You
> should get "kylin.metrics.prefix" from kylin.properties. If this item is set,
> take the value set by the user, if not set, go to the default value. "KYLIN"
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
