chore(UUID): Streamline UUID implementation #851

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.

Sign up for GitHub

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

Jump to bottom

Merged

kodiakhq merged 1 commit into main from chore/uuid/impl

May 8, 2023

Contributor

candiduslynx commented May 8, 2023

Based on apache/arrow#35457 review


          streamline UUID impl based on the upstream review

e25a225

candiduslynx added the automerge label

candiduslynx requested a review from yevgenypats as a code owner

May 8, 2023 16:29

github-actions bot added the ignore-for-release label

yevgenypats approved these changes

View reviewed changes

kodiakhq bot merged commit 7fe9990 into main

kodiakhq bot deleted the chore/uuid/impl branch

May 8, 2023 16:33

github-actions bot commented May 8, 2023

⏱️ Benchmark results

DefaultConcurrencyDFS-2 resources/s: 10,281
DefaultConcurrencyRoundRobin-2 resources/s: 12,208
Glob-2 ns/op: 269.2
TablesWithChildrenDFS-2 resources/s: 25,810
TablesWithChildrenRoundRobin-2 resources/s: 22,680
TablesWithRateLimitingDFS-2 resources/s: 28.39
TablesWithRateLimitingRoundRobin-2 resources/s: 854.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels