alpinegizmo commented on code in PR #27841: URL: https://github.com/apache/flink/pull/27841#discussion_r3018217627
########## docs/content/docs/deployment/filesystems/s3.md: ########## @@ -108,7 +108,7 @@ presto.s3.credentials-provider: org.apache.flink.fs.s3.common.token.DynamicTempo ### Configure Non-S3 Endpoint -The S3 filesystems also support using S3 compliant object stores such as [IBM's Cloud Object Storage](https://www.ibm.com/cloud/object-storage) and [MinIO](https://min.io/). +The S3 filesystems also support using S3 compliant object stores such as [IBM's Cloud Object Storage](https://www.ibm.com/cloud/object-storage) and [Cloudflare R2](https://developers.cloudflare.com/r2/). Review Comment: It feels a bit off to only mention a couple of the proprietary offerings here, and none of the free, open source options. Given there are so many offerings, what if we avoid appearing to endorse any specific solutions and simply say this: The S3 filesystems also support using S3 compliant object stores, including both open source and commercial offerings. -- 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]
