jorisvandenbossche commented on code in PR #41494: URL: https://github.com/apache/arrow/pull/41494#discussion_r1587650798
########## ci/scripts/python_build.sh: ########## @@ -58,7 +58,7 @@ fi export PYARROW_CMAKE_GENERATOR=${CMAKE_GENERATOR:-Ninja} export PYARROW_BUILD_TYPE=${CMAKE_BUILD_TYPE:-debug} -export PYARROW_WITH_ACERO=${ARROW_ACERO:-OFF} +export PYARROW_WITH_ACERO=${ARROW_ACERO:-ON} Review Comment: Dataset is on by default (a few lines below), and so that in practice means Acero is on by default as well. Now, I changed this in the CMakeLists file to do that automatically, so I could also leave it here as is, but this seems more correct -- 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-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org