Skip to content

Conversation

@domoritz
Copy link
Member

@domoritz domoritz commented Mar 22, 2024

Simpler version of #40712 that preserves get.

@domoritz domoritz merged commit 7d18448 into apache:main Apr 16, 2024
@domoritz domoritz deleted the dom/at-2 branch April 16, 2024 20:19
@conbench-apache-arrow
Copy link

After merging your PR, Conbench analyzed the 7 benchmarking runs that have been run so far on merge-commit 7d18448.

There were 6 benchmark results indicating a performance regression:

The full Conbench report has more details. It also includes information about 1 possible false positive for unstable benchmarks that are known to sometimes produce them.

domoritz added a commit to domoritz/arrow that referenced this pull request Apr 17, 2024
Simpler version of apache#40712 that
preserves `get`.
* GitHub Issue: apache#39131
@domoritz domoritz removed the awaiting committer review Awaiting committer review label Apr 18, 2024
raulcd pushed a commit that referenced this pull request Apr 29, 2024
Simpler version of #40712 that
preserves `get`.
* GitHub Issue: #39131
vibhatha pushed a commit to vibhatha/arrow that referenced this pull request May 25, 2024
@wgtmac wgtmac removed their assignment Jul 12, 2025
wgtmac added a commit to wgtmac/arrow that referenced this pull request Oct 12, 2025
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.

3 participants