devmadhuu commented on code in PR #11229: URL: https://github.com/apache/ozone/pull/11229#discussion_r4119600535
########## hadoop-ozone/s3gateway/src/main/java/org/apache/hadoop/ozone/s3/util/S3StorageClass.java: ########## @@ -0,0 +1,101 @@ +/* Review Comment: Thanks @greenwich for raising the linked-bucket case. I traced the current HDDS-11233 flow. Although the low-level `OzoneManagerUtils.getBucketInfo()` returns the raw link row, the S3 client path goes through `OzoneManager.getBucketInfo()`, which calls [enrichLinkBucketInfo()](https://github.com/apache/ozone/blob/5ecb63ab66d8e65c6962fd3f12369f72d82c191a/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/OzoneManager.java#L3070). That method resolves the real target and `withOperationalPropertiesFrom()` copies both `storagePolicy` and `defaultReplicationConfig` from the target while preserving the link identity. The OM key-write path then resolves to the same target bucket. Therefore, the target bucket is the source of both policy and replication defaults. I’ll add an S3 linked-bucket regression test when the storage-class mapping is connected to the write path in the follow-up PR. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
