Skip to content

[benchmark] Fix copy-paste labels in micro-benchmarks#8521

Merged
JingsongLi merged 1 commit into
apache:masterfrom
thswlsqls:fix/benchmark-copy-paste-labels
Jul 9, 2026
Merged

[benchmark] Fix copy-paste labels in micro-benchmarks#8521
JingsongLi merged 1 commit into
apache:masterfrom
thswlsqls:fix/benchmark-copy-paste-labels

Conversation

@thswlsqls

Copy link
Copy Markdown
Contributor

Purpose

  • Correct two non-behavioral copy-paste labels in paimon-micro-benchmarks.
  • LookupReaderBenchmark Javadoc described writing; it is a reader benchmark, so the wording now says reading.
  • ArrowWriteBenchmark#testWrite created new Benchmark("read", ...), which the harness prints; the benchmark name is now write.

Tests

  • Typo/wording fix, no behavior change — no test added. mvn spotless:check -pl paimon-benchmark/paimon-micro-benchmarks passed.
  • mvn -pl paimon-benchmark/paimon-micro-benchmarks -DskipTests clean test-compile passed; 17 test sources compiled. Benchmarks were not executed because each @Test runs a long benchmark.

@JingsongLi JingsongLi merged commit a16880e into apache:master Jul 9, 2026
12 checks passed
@thswlsqls thswlsqls deleted the fix/benchmark-copy-paste-labels branch July 11, 2026 00:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants