Skip to content

[CASSANDRA-21388][trunk] Reduce microbenchmark smoke tests execution#4827

Closed
netudima wants to merge 1 commit into
apache:trunkfrom
netudima:CASSANDRA-21388-trunk
Closed

[CASSANDRA-21388][trunk] Reduce microbenchmark smoke tests execution#4827
netudima wants to merge 1 commit into
apache:trunkfrom
netudima:CASSANDRA-21388-trunk

Conversation

@netudima
Copy link
Copy Markdown
Contributor

@netudima netudima commented May 19, 2026

  • set explicitly duration for an iteration to 1 sec
  • reduce number of option combinations in tests (all the details are not so needed for a regression testing to see if the benchmarks are still alive)
  • fix the tests which do not release resources properly and JMH waits for a timeout to kill a forked JVM
  • support splits for microbench-test

CASSANDRA-21388

@netudima netudima changed the title [][trunk] Reduce microbenchmark smoke tests execution [CASSANDRA-21388][trunk] Reduce microbenchmark smoke tests execution May 19, 2026
Comment thread .build/build-bench.xml Outdated
@netudima netudima force-pushed the CASSANDRA-21388-trunk branch from 4468e54 to 34e1668 Compare May 21, 2026 17:21
Copy link
Copy Markdown
Member

@michaelsembwever michaelsembwever left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

Set 1 thread and 1 second for the measurement iteration (note: it is a smoke test only, we do not analyze results from such short measurements)
Exclude some options for heavy benchmarks by default
Shutdown all threads once JMH test is completed to avoid awaiting of a forked JVM till timeout at the end

patch by Dmitry Konstantinov; reviewed by Michael Semb Wever for CASSANDRA-21388
@netudima netudima force-pushed the CASSANDRA-21388-trunk branch from 34e1668 to 6dd9e97 Compare May 21, 2026 19:26
@netudima
Copy link
Copy Markdown
Contributor Author

Merged as 1f078c0

@netudima netudima closed this May 21, 2026
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