You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@kafka.apache.org by "ASF GitHub Bot (Jira)" <ji...@apache.org> on 2020/04/16 13:53:00 UTC
[jira] [Commented] (KAFKA-9881) Verify whether RocksDBMetrics Get
Measurements from RocksDB in a Unit Test instead of an Intergration Test
[ https://issues.apache.org/jira/browse/KAFKA-9881?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17084881#comment-17084881 ]
ASF GitHub Bot commented on KAFKA-9881:
---------------------------------------
cadonna commented on pull request #8501: KAFKA-9881: Convert integration test to verify measurements from RocksDB to unit test
URL: https://github.com/apache/kafka/pull/8501
The integration test RocksDBMetricsIntegrationTest takes pretty long to complete.
Most of the runtime is spent in the two tests that verify whether the RocksDB
metrics get actual measurements from RocksDB. Those tests need to wait for the thread
that collects the measurements of the RocksDB metrics to trigger the first recordings
of the metrics.
This PR adds a unit test that verifies whether the Kafka Streams metrics get the
measurements from RocksDB and removes the two integration tests that verified it
before. The verification of the creation and scheduling of the RocksDB metrics
recording trigger thread is already contained in KafkaStreamsTest and consequently
it is not part of this PR.
### Committer Checklist (excluded from commit message)
- [ ] Verify design and implementation
- [ ] Verify test coverage and CI build status
- [ ] Verify documentation (including upgrade notes)
----------------------------------------------------------------
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
> Verify whether RocksDBMetrics Get Measurements from RocksDB in a Unit Test instead of an Intergration Test
> ----------------------------------------------------------------------------------------------------------
>
> Key: KAFKA-9881
> URL: https://issues.apache.org/jira/browse/KAFKA-9881
> Project: Kafka
> Issue Type: Improvement
> Components: streams
> Affects Versions: 2.5.0
> Reporter: Bruno Cadonna
> Priority: Major
>
> The integration test {{RocksDBMetricsIntegrationTest}} takes pretty long to complete. The main part of the runtime is spent in the two tests that verify whether the rocksDB metrics get actual measurements from RocksDB. Those tests need to wait for the thread that collects the measurements of the RocksDB metrics to trigger the first recordings of the metrics. These tests do not need to run as integration tests and thus they shall be converted into unit tests to save runtime.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)