Skip to content

Conversation

alekkol
Copy link
Contributor

@alekkol alekkol commented Jul 31, 2025

Summary

Added 2 benchmark to demostrate performance regression between v1 and v2 versions.

Relevant issue: #2516

@CLAassistant
Copy link

CLAassistant commented Jul 31, 2025

CLA assistant check
All committers have signed the CLA.

@chernser
Copy link
Contributor

@alekkol
I've approved. Is it fine to merge?

@chernser chernser marked this pull request as ready for review August 18, 2025 15:05
@chernser chernser merged commit 15eda02 into ClickHouse:main Aug 18, 2025
17 of 23 checks passed
@alekkol
Copy link
Contributor Author

alekkol commented Sep 8, 2025

@chernser Sure. I provided this PR as a reproducer for the performance issue. If you find it useful to have this code in the mainline, I have no objections :)

@chernser
Copy link
Contributor

chernser commented Sep 8, 2025

@alekkol
I've merged it. Thank you!

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