Added optional flag for building Arrow with S3 filesystem support

[jdye64]

Added the cmake build flag CUDF_ENABLE_ARROW_S3 to conditionally build Arrow with S3 filesystem support. By default support for S3 filesystem will be enabled. If S3 support needs to be disabled you can pass -DCUDF_ENABLE_ARROW_S3=OFF

[jlowe]

Java approval

[robertmaynard]

What happens when S3 is disabled and arrow_io_source is called with an S3 backed URI? Should we have a test that we get the expected exception and don't crash?

[jdye64]

What happens when S3 is disabled and arrow_io_source is called with an S3 backed URI? Should we have a test that we get the expected exception and don't crash?

In that case https://github.com/rapidsai/cudf/blob/branch-21.08/cpp/include/cudf/io/datasource.hpp#L321 would trigger. Always good to add a test however.

[codecov]

Codecov Report

Merging #8531 (262d463) into branch-21.08 (93ce6c7) will not change coverage.
The diff coverage is n/a.

❗ Current head 262d463 differs from pull request most recent head 20b2a4f. Consider uploading reports for the commit 20b2a4f to get more accurate results

@@              Coverage Diff              @@
##           branch-21.08    #8531   +/-   ##
=============================================
  Coverage         82.57%   82.57%           
=============================================
  Files               109      109           
  Lines             17870    17870           
=============================================
  Hits              14757    14757           
  Misses             3113     3113           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 93ce6c7...20b2a4f. Read the comment docs.

[trxcllnt]

@gpucibot merge