Skip to content

Remove jmh from CI and check - #964

Merged
snazy merged 1 commit into
projectnessie:mainfrom
snazy:no-jmh
Aug 20, 2026
Merged

Remove jmh from CI and check#964
snazy merged 1 commit into
projectnessie:mainfrom
snazy:no-jmh

Conversation

@snazy

@snazy snazy commented Aug 20, 2026

Copy link
Copy Markdown
Member

Do not run JMH in CI, it's results are not monitored and JMH results from GH hosted runners aren't reliable either.

Also remove JMH invocation from Gradle's check.

Do not run JMH in CI, it's results are not monitored and JMH results from GH hosted runners aren't reliable either.

Also remove JMH invocation from Gradle's `check`.
@snazy
snazy requested a review from XN137 August 20, 2026 09:51
@snazy

snazy commented Aug 20, 2026

Copy link
Copy Markdown
Member Author

JMH takes quite a while now and will, with the upcoming changes, get more benchmarks.
Leaving JMH runs for manual invocation.

Comment thread .github/workflows/main.yml
@snazy
snazy requested a review from XN137 August 20, 2026 10:10
@snazy
snazy merged commit 241439d into projectnessie:main Aug 20, 2026
3 checks passed
@snazy
snazy deleted the no-jmh branch August 20, 2026 10:29
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