Skip to content

Support for selectable hashmap implementation in the combiner_perf #18010

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
May 5, 2025

Conversation

pzuev
Copy link
Collaborator

@pzuev pzuev commented May 5, 2025

Also:

  • change the timing method (use the thread CPU time)
  • fix bar colors in the result chart, remove the "bytes per key" column from the formatted table

Changelog category

  • Not for changelog (changelog entry is not required)

@pzuev pzuev marked this pull request as ready for review May 5, 2025 10:58
@pzuev pzuev requested a review from a team as a code owner May 5, 2025 10:58
Copy link

github-actions bot commented May 5, 2025

2025-05-05 10:59:20 UTC Pre-commit check linux-x86_64-release-asan for 66cff16 has started.
2025-05-05 10:59:40 UTC Artifacts will be uploaded here
2025-05-05 11:02:39 UTC ya make is running...
🟢 2025-05-05 11:03:40 UTC Tests successful.

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
0 0 0 0 0 0

🟢 2025-05-05 11:03:46 UTC Build successful.

Copy link

github-actions bot commented May 5, 2025

🟢 2025-05-05 11:02:08 UTC The validation of the Pull Request description is successful.

@pzuev pzuev enabled auto-merge (squash) May 5, 2025 11:06
Copy link

github-actions bot commented May 5, 2025

2025-05-05 11:39:02 UTC Pre-commit check linux-x86_64-relwithdebinfo for 66cff16 has started.
2025-05-05 11:39:25 UTC Artifacts will be uploaded here
2025-05-05 11:42:44 UTC ya make is running...
🟢 2025-05-05 11:43:59 UTC Tests successful.

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
0 0 0 0 0 0

🟢 2025-05-05 11:44:04 UTC Build successful.

@pzuev pzuev merged commit 8e93406 into ydb-platform:main May 5, 2025
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants