nealrichardson commented on PR #14235: URL: https://github.com/apache/arrow/pull/14235#issuecomment-1346788804
This feels like a really big change to support something that practically no one has (how many people are building aws-sdk-cpp separately from source--it's not available in package managers so that's what this entails, right?--and then trying to install the R package?). But we don't build with ARROW_S3 on CRAN so I guess the risk there is lower. My other concern is that it now looks like we have all of these other system dependencies, and that we're no longer testing more stripped down environments that don't have all of these libraries installed. Do I read that correctly? That said, I trust Kou's judgment, so I won't get in the way of merging. -- 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
