[
https://issues.apache.org/jira/browse/KYLIN-5029?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17386981#comment-17386981
]
ASF GitHub Bot commented on KYLIN-5029:
---------------------------------------
codecov-commenter commented on pull request #1683:
URL: https://github.com/apache/kylin/pull/1683#issuecomment-886306388
#
[Codecov](https://codecov.io/gh/apache/kylin/pull/1683?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
Report
> :exclamation: No coverage uploaded for pull request base
(`kylin-on-parquet-v2@a9ad60a`). [Click here to learn what that
means](https://docs.codecov.io/docs/error-reference?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#section-missing-base-commit).
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/kylin/pull/1683?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
```diff
@@ Coverage Diff @@
## kylin-on-parquet-v2 #1683 +/- ##
======================================================
Coverage ? 24.61%
Complexity ? 4832
======================================================
Files ? 1162
Lines ? 66519
Branches ? 9621
======================================================
Hits ? 16376
Misses ? 48407
Partials ? 1736
```
------
[Continue to review full report at
Codecov](https://codecov.io/gh/apache/kylin/pull/1683?src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
> **Legend** - [Click here to learn
more](https://docs.codecov.io/docs/codecov-delta?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by
[Codecov](https://codecov.io/gh/apache/kylin/pull/1683?src=pr&el=footer&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
Last update
[a9ad60a...7ee2a70](https://codecov.io/gh/apache/kylin/pull/1683?src=pr&el=lastupdated&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
Read the [comment
docs](https://docs.codecov.io/docs/pull-request-comments?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
> metrics test may fail because the cache is not refreshed
> --------------------------------------------------------
>
> Key: KYLIN-5029
> URL: https://issues.apache.org/jira/browse/KYLIN-5029
> Project: Kylin
> Issue Type: Improvement
> Components: Tools, Build and Test
> Reporter: Congling Xia
> Assignee: Congling Xia
> Priority: Major
>
> Tests in _org.apache.kylin.rest.metrics.QueryMetricsTest_ may fail in current
> CI processes. There are some "Thread.sleep(..)" in the test code to wait for
> metrics cache being refreshed.
> After digging into the Hadoop metrics implementation, I find that the default
> refreshing interval is 10 seconds. So test thread should sleep for more than
> 10 seconds to may sure metrics are up-to-date. I checked the sleep intervals
> in the test code and they are not greater than 10000 (ms). That's why CI
> processes fail accidentally.
> It is better to configure metrics refreshing interval to a smaller value.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
