Presto support COS select

[Coderlxl]

We used Presto to test COS Select feature on csv files, but it‘s failed.

We checked Presto source code and COS documentation, found that it was caused by the following reasons:

1. COS Select feature only support virtual-hosted style URL. But Presto only support path-style URL. The hard code is in PrestoS3ClientFactory class.
   

2. Cos Select feature don't support csv comments = "\uFDD0", but it's hard code in S3SelectCsvRecordReader class.
   

So, I think these parameters should be configurable in Presto. My patch fixed this problem.
0001-support-cos-select.patch.txt

Test fixed result:
(1) Fix Before, Presto's hive.properties is:
hive.s3.aws-access-key=AK
hive.s3.aws-secret-key=SK
hive.s3.endpoint=https://cos.ap-beijing.myqcloud.com
hive.s3select-pushdown.enabled=true
#hive.s3.path-style-access=false
#hive.s3.csv-comments=#

failed result

(2) Fix After, Presto's hive.properties is:
hive.s3.aws-access-key=AK
hive.s3.aws-secret-key=SK
hive.s3.endpoint=https://cos.ap-beijing.myqcloud.com
hive.s3select-pushdown.enabled=true
hive.s3.path-style-access=false
hive.s3.csv-comments=#

succeed result

COS Reference Documentation https://intl.cloud.tencent.com/document/product/436/32472

[electrum]

Thanks for filing this issue and including a patch. Can you sign the CLA and submit this patch as a pull request?

[hashhar]

s3 select was removed in #18241 and I don't think we'll be adding COS Select because of the direction filesystem code is moving in.