spuru9 commented on code in PR #28076:
URL: https://github.com/apache/flink/pull/28076#discussion_r3170058489


##########
docs/content.zh/docs/deployment/filesystems/s3.md:
##########
@@ -170,27 +170,7 @@ cp ./opt/flink-s3-fs-native-{{< version >}}.jar 
./plugins/s3-fs-native/
 
 In addition to the [common configuration](#common-configuration) options 
(`s3.access-key`, `s3.secret-key`, `s3.endpoint`, `s3.path-style-access`), the 
Native S3 FileSystem supports the following options:
 
-```yaml
-# Server-side encryption
-s3.sse.type: sse-s3         # or sse-kms, aws:kms, AES256, none (default)
-s3.sse.kms.key-id: arn:aws:kms:region:account:key/id   # Required for SSE-KMS
-
-# IAM role assumption for cross-account access
-s3.assume-role.arn: arn:aws:iam::account:role/RoleName
-s3.assume-role.external-id: external-id-if-required
-s3.assume-role.session-name: flink-s3-session
-s3.assume-role.session-duration: 3600
-
-# Performance tuning
-s3.upload.min.part.size: 5242880        # 5 MB default
-s3.upload.max.concurrent.uploads: 4     # Based on CPU cores
-s3.read.buffer.size: 262144             # 256 KB default
-s3.async.enabled: true                  # Async read/write operations
-s3.bulk-copy.enabled: true              # Bulk copy via S3TransferManager
-s3.bulk-copy.max-concurrent: 16         # Max concurrent copy ops
-```
-
-When `fs.s3.aws.credentials.provider` is not set, the Native S3 FileSystem 
automatically builds a credentials chain in the following order: delegation 
tokens, static credentials (if `s3.access-key` and `s3.secret-key` are 
configured), and the AWS SDK v2 `DefaultCredentialsProvider` (environment 
variables, instance profiles, etc.). You only need to set this option if you 
require a custom provider chain.
+View the detailed configuration at 
[native-s3-fs](https://github.com/apache/flink/tree/master/flink-filesystems/flink-s3-fs-native)

Review Comment:
   nit: for someone opening the configuration, 
https://github.com/apache/flink/tree/master/flink-filesystems/flink-s3-fs-native#configuration-options
 might be a better place to link. 
   Work both ways, just a food for thought. 



-- 
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]

Reply via email to