[
https://issues.apache.org/jira/browse/KYLIN-4855?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17258203#comment-17258203
]
ASF GitHub Bot commented on KYLIN-4855:
---------------------------------------
codecov-io commented on pull request #1536:
URL: https://github.com/apache/kylin/pull/1536#issuecomment-753974549
# [Codecov](https://codecov.io/gh/apache/kylin/pull/1536?src=pr&el=h1) Report
> Merging
[#1536](https://codecov.io/gh/apache/kylin/pull/1536?src=pr&el=desc) (be98fd8)
into
[master](https://codecov.io/gh/apache/kylin/commit/47f8e54558d885f384f5d98c419cc1fa532bca7b?el=desc)
(47f8e54) will **decrease** coverage by `0.09%`.
> The diff coverage is `1.23%`.
[](https://codecov.io/gh/apache/kylin/pull/1536?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #1536 +/- ##
============================================
- Coverage 25.51% 25.41% -0.10%
+ Complexity 6765 6763 -2
============================================
Files 1505 1508 +3
Lines 93632 93918 +286
Branches 13117 13158 +41
============================================
- Hits 23886 23872 -14
- Misses 67370 67664 +294
- Partials 2376 2382 +6
```
| [Impacted
Files](https://codecov.io/gh/apache/kylin/pull/1536?src=pr&el=tree) | Coverage
Δ | Complexity Δ | |
|---|---|---|---|
|
[.../java/org/apache/kylin/common/KylinConfigBase.java](https://codecov.io/gh/apache/kylin/pull/1536/diff?src=pr&el=tree#diff-Y29yZS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL2NvbW1vbi9LeWxpbkNvbmZpZ0Jhc2UuamF2YQ==)
| `12.46% <0.00%> (-0.05%)` | `51.00 <0.00> (ø)` | |
|
[...n/java/org/apache/kylin/common/util/ClassUtil.java](https://codecov.io/gh/apache/kylin/pull/1536/diff?src=pr&el=tree#diff-Y29yZS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL2NvbW1vbi91dGlsL0NsYXNzVXRpbC5qYXZh)
| `49.20% <0.00%> (-2.47%)` | `9.00 <0.00> (ø)` | |
|
[.../java/org/apache/kylin/job/util/JobRestClient.java](https://codecov.io/gh/apache/kylin/pull/1536/diff?src=pr&el=tree#diff-Y29yZS1qb2Ivc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL2pvYi91dGlsL0pvYlJlc3RDbGllbnQuamF2YQ==)
| `0.00% <0.00%> (ø)` | `0.00 <0.00> (?)` | |
|
[...che/kylin/engine/spark/SparkMergingDictionary.java](https://codecov.io/gh/apache/kylin/pull/1536/diff?src=pr&el=tree#diff-ZW5naW5lLXNwYXJrL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9reWxpbi9lbmdpbmUvc3BhcmsvU3BhcmtNZXJnaW5nRGljdGlvbmFyeS5qYXZh)
| `0.00% <0.00%> (ø)` | `0.00 <0.00> (ø)` | |
|
[...g/apache/kylin/query/relnode/OLAPAggregateRel.java](https://codecov.io/gh/apache/kylin/pull/1536/diff?src=pr&el=tree#diff-cXVlcnkvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL3F1ZXJ5L3JlbG5vZGUvT0xBUEFnZ3JlZ2F0ZVJlbC5qYXZh)
| `0.00% <0.00%> (ø)` | `0.00 <0.00> (ø)` | |
|
[.../org/apache/kylin/query/relnode/OLAPFilterRel.java](https://codecov.io/gh/apache/kylin/pull/1536/diff?src=pr&el=tree#diff-cXVlcnkvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL3F1ZXJ5L3JlbG5vZGUvT0xBUEZpbHRlclJlbC5qYXZh)
| `0.00% <0.00%> (ø)` | `0.00 <0.00> (ø)` | |
|
[.../org/apache/kylin/query/relnode/OLAPWindowRel.java](https://codecov.io/gh/apache/kylin/pull/1536/diff?src=pr&el=tree#diff-cXVlcnkvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL3F1ZXJ5L3JlbG5vZGUvT0xBUFdpbmRvd1JlbC5qYXZh)
| `0.00% <0.00%> (ø)` | `0.00 <0.00> (ø)` | |
|
[.../query/relnode/visitor/TupleExpressionVisitor.java](https://codecov.io/gh/apache/kylin/pull/1536/diff?src=pr&el=tree#diff-cXVlcnkvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL3F1ZXJ5L3JlbG5vZGUvdmlzaXRvci9UdXBsZUV4cHJlc3Npb25WaXNpdG9yLmphdmE=)
| `0.00% <0.00%> (ø)` | `0.00 <0.00> (ø)` | |
|
[...torage/hbase/steps/HBaseFlinkOutputTransition.java](https://codecov.io/gh/apache/kylin/pull/1536/diff?src=pr&el=tree#diff-c3RvcmFnZS1oYmFzZS9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUva3lsaW4vc3RvcmFnZS9oYmFzZS9zdGVwcy9IQmFzZUZsaW5rT3V0cHV0VHJhbnNpdGlvbi5qYXZh)
| `0.00% <0.00%> (ø)` | `0.00 <0.00> (ø)` | |
|
[...che/kylin/storage/hbase/steps/HBaseFlinkSteps.java](https://codecov.io/gh/apache/kylin/pull/1536/diff?src=pr&el=tree#diff-c3RvcmFnZS1oYmFzZS9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUva3lsaW4vc3RvcmFnZS9oYmFzZS9zdGVwcy9IQmFzZUZsaW5rU3RlcHMuamF2YQ==)
| `0.00% <0.00%> (ø)` | `0.00 <0.00> (ø)` | |
| ... and [19
more](https://codecov.io/gh/apache/kylin/pull/1536/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at
Codecov](https://codecov.io/gh/apache/kylin/pull/1536?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/1536?src=pr&el=footer). Last
update
[5c12d08...be98fd8](https://codecov.io/gh/apache/kylin/pull/1536?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)
