You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by "jorisvandenbossche (via GitHub)" <gi...@apache.org> on 2023/05/16 09:00:44 UTC

[GitHub] [arrow] jorisvandenbossche commented on pull request #35565: GH-35498: [C++] Relax EnsureAlignment check in Acero from requiring 64-byte aligned buffers to requiring value-aligned buffers

jorisvandenbossche commented on PR #35565:
URL: https://github.com/apache/arrow/pull/35565#issuecomment-1549273891

   > I'm running the benchmarks again but, as best I can tell, these regressions are noise, though it is quite difficult to say for sure.
   
   I am not sure why it doesn't show up on the landing page linked from the bot comment (https://conbench.ursa.dev/compare/runs/80cbe13b10ca4d39b05e59e4b4d5037d...6eab503c9deb4e8ba81b02093e4604dc/), but looking at some of the individual impacted benchmarks, they seem to show a good speed-up for this specific run. For example: 
   https://conbench.ursa.dev/benchmark-results/0c38028ec8c54423901ab411f9a523ca/
   https://conbench.ursa.dev/benchmark-results/e95959fb9592435786916f29ae11f13c/
   https://conbench.ursa.dev/benchmark-results/d9f2511dd5854eff958bd3d7627b8b60/
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org