Skip to content

Conversation

@frankvicky
Copy link
Contributor

JIRA: KAFKA-17569

Committer Checklist (excluded from commit message)

  • Verify design and implementation
  • Verify test coverage and CI build status
  • Verify documentation (including upgrade notes)

@github-actions github-actions bot added core Kafka Broker performance build Gradle build or GitHub Actions labels Nov 10, 2024
@frankvicky
Copy link
Contributor Author

frankvicky commented Nov 10, 2024

Test with --bytes 100000000 --size 10 --files 1 --channel 1

> Task :jmh-benchmarks:org.apache.kafka.jmh.log.TestLinearWriteSpeed.main()
    mb_sec	avg_latency	max_latency
    22.470	     0.000	     0.406
    22.341	     0.000	     0.124
    22.296	     0.000	     0.113
    21.921	     0.000	     0.111
22.21302268753944 MB per sec

Test with --bytes 100000000 --size 4096 --files 1 --channel 1

> Task :jmh-benchmarks:org.apache.kafka.jmh.log.TestLinearWriteSpeed.main()
    mb_sec	avg_latency	max_latency
2187.074699536301 MB per sec

@chia7712 chia7712 merged commit 0e4d8b3 into apache:trunk Nov 26, 2024
6 checks passed
chiacyu pushed a commit to chiacyu/kafka that referenced this pull request Nov 30, 2024
Reviewers: Chia-Ping Tsai <chia7712@gmail.com>
tedyu pushed a commit to tedyu/kafka that referenced this pull request Jan 6, 2025
Reviewers: Chia-Ping Tsai <chia7712@gmail.com>
chia7712 pushed a commit that referenced this pull request Aug 17, 2025
This PR fixes a problem related to `TestLinearWriteSpeed`.  During my
work on KIP-780, I discovered that benchmarks for `TestLinearWriteSpeed`
do not account for compression algorithms. It always uses
`Compression.NONE` when creating records. The problem was introduced in
this PR [1].

[1] - #17736

Reviewers: Ken Huang <s7133700@gmail.com>, Mickael Maison
<mickael.maison@gmail.com>, Chia-Ping Tsai <chia7712@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build Gradle build or GitHub Actions ci-approved core Kafka Broker performance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants