xhochy commented on a change in pull request #11916:
URL: https://github.com/apache/arrow/pull/11916#discussion_r791530051



##########
File path: dev/tasks/conda-recipes/arrow-cpp/build-pyarrow.sh
##########
@@ -23,6 +23,7 @@ export PYARROW_WITH_PARQUET=1
 export PYARROW_WITH_PLASMA=1
 export PYARROW_WITH_S3=1
 export PYARROW_CMAKE_GENERATOR=Ninja
+export PYARROW_CMAKE_OPTIONS="-DARROW_SIMD_LEVEL=NONE"

Review comment:
       Two things:
   
   - The ppc64le and aarch64 builds somehow try to detect SSE4 and fail as the 
compiler doesn't support it. There is something wrong with the 
   - Otherwise, we probably can set that to SSE2 in the case of x64 but that is 
probably already part of NONE? conda-forge packages are built for the baseline 
of CPUs and thus using a higher SIMD level would break them for some users. 
There is work on supporting instruction set dependent installation in conda but 
it is not fully done yet.

##########
File path: dev/tasks/conda-recipes/arrow-cpp/meta.yaml
##########
@@ -81,6 +82,7 @@ outputs:
         - lz4-c
         - numpy
         - orc  # [unix]
+        - openssl

Review comment:
       Yes, but Gandiva also links directly to it and thus it should also be 
listed directly here to also get rebuilds for OpenSSL version updates.

##########
File path: dev/tasks/conda-recipes/arrow-cpp/build-pyarrow.sh
##########
@@ -23,6 +23,7 @@ export PYARROW_WITH_PARQUET=1
 export PYARROW_WITH_PLASMA=1
 export PYARROW_WITH_S3=1
 export PYARROW_CMAKE_GENERATOR=Ninja
+export PYARROW_CMAKE_OPTIONS="-DARROW_SIMD_LEVEL=NONE"

Review comment:
       Looking at the values of `ARROW_SIMD_LEVEL` `NONE` is correct for the 
conda-forge case. We cannot build packages with `SSE4_2` by default as there 
are still (supported) users without it. I've opened an issue at 
https://issues.apache.org/jira/browse/ARROW-15445 With the workaround here, 
we're fine for now but sadly https://issues.apache.org/jira/browse/ARROW-15444 
is an issues that should block the release as especially the CUDA builds are 
used by several downstream users.




-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to