You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2021/07/19 21:16:10 UTC

[GitHub] [arrow] lidavidm edited a comment on pull request #10750: ARROW-13382: [C++] Avoid multiple definitions of same symbol

lidavidm edited a comment on pull request #10750:
URL: https://github.com/apache/arrow/pull/10750#issuecomment-882865286


   1. Sum happens to be one of the few functions with explicit kernels for differing SIMD levels, and it accomplishes this with some common code. In the future, hopefully we'll have a better approach for SIMD, but this sort of thing will be something we need to keep in mind.
   2. The base commit here is the original commit that broke the sum kernel, so yes, before the one that broke with zstd.


-- 
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