Skip to content

Use JMH for benchmarks.#228

Closed
shawjef3 wants to merge 1 commit intomaxmind:mainfrom
shawjef3:jmh
Closed

Use JMH for benchmarks.#228
shawjef3 wants to merge 1 commit intomaxmind:mainfrom
shawjef3:jmh

Conversation

@shawjef3
Copy link

@shawjef3 shawjef3 commented Feb 7, 2025

This is for #227.

@oschwald
Copy link
Member

oschwald commented Feb 7, 2025

Although I am not opposed to using JMH for benchmarks, I do not want to reorganize the repo to do so. I believe we could keep the existing repo structure and scope the benchmarks to test.

@shawjef3 shawjef3 mentioned this pull request Feb 8, 2025
@shawjef3
Copy link
Author

shawjef3 commented Feb 8, 2025

I thought so, too. I put effort into that, and found it wasn't easily possible. I had difficulty getting the shadow jar plugin to make a jar that included the tests. It was easier for me to make it a separate module.

@shawjef3 shawjef3 closed this Feb 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants