raulcd commented on code in PR #50195:
URL: https://github.com/apache/arrow/pull/50195#discussion_r3956722124
##########
cpp/cmake_modules/DefineOptions.cmake:
##########
@@ -396,17 +396,11 @@ takes precedence over ccache if a storage backend is
configured" ON)
ARROW_JSON)
define_option(ARROW_S3
- "Build Arrow with S3 support (requires the AWS SDK for C++)"
+ "Build Arrow S3 Module (requires the AWS SDK for C++)"
OFF
DEPENDS
ARROW_FILESYSTEM)
Review Comment:
We use the term Module for `Build the Arrow Acero Engine Module`, `Build the
Arrow CSV Parser Module`, or `Build the Arrow Dataset Modules`. I think this
is just fine with our current use of terminology.
--
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]
