Skip to content

Conversation

@jaydeluca
Copy link
Collaborator

Tested on my fork, results can be seen here: https://github.com/jaydeluca/client_java/tree/benchmarks

Signed-off-by: Jay DeLuca <jaydeluca4@gmail.com>
Signed-off-by: Jay DeLuca <jaydeluca4@gmail.com>
Signed-off-by: Jay DeLuca <jaydeluca4@gmail.com>
Signed-off-by: Jay DeLuca <jaydeluca4@gmail.com>
Signed-off-by: Jay DeLuca <jaydeluca4@gmail.com>
@jaydeluca jaydeluca requested a review from zeitlinger January 25, 2026 20:52
Signed-off-by: Jay DeLuca <jaydeluca4@gmail.com>
Signed-off-by: Jay DeLuca <jaydeluca4@gmail.com>
Copy link
Member

@zeitlinger zeitlinger left a comment

Choose a reason for hiding this comment

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

great!


- name: Generate benchmark summary
run: |
python3 .mise/tasks/generate_benchmark_summary.py \
Copy link
Member

Choose a reason for hiding this comment

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

I think this should also work as mise run generate_benchmark_summary

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

updated to use a mise task


- name: Run JMH benchmarks
run: |
# 3 forks, 3 warmup, 5 measurement iterations (~60 min total)
Copy link
Member

Choose a reason for hiding this comment

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

I see you added mise commands - maybe also make this one

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

done

Signed-off-by: Jay DeLuca <jaydeluca4@gmail.com>
@zeitlinger zeitlinger merged commit 0569554 into main Jan 26, 2026
37 checks passed
@zeitlinger zeitlinger deleted the test-benchmarks branch January 26, 2026 19:22
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.

3 participants