Re: [PR] HDDS-15925. Skip redundant InfoBucket RPC on OFS getFileStatus with a bounded bucket-layout cache [ozone]

["Copilot (via GitHub)" <gi...@apache.org>]

Copilot commented on code in PR #11176:
URL: https://github.com/apache/ozone/pull/11176#discussion_r3962737045


##########
hadoop-ozone/ozonefs-common/src/main/java/org/apache/hadoop/fs/ozone/BasicRootedOzoneClientAdapterImpl.java:
##########
@@ -353,6 +367,39 @@ private OzoneBucket getBucket(String volumeStr, String bucketStr,
     return bucket;
   }
 
+  /**
+   * Returns the resolved bucket layout for the given path, caching it so that
+   * repeated getFileStatus calls on the same bucket avoid the InfoBucket RPC.
+   * On a cache miss the layout is fetched via {@link ClientProtocol
+   * #getBucketDetails} and, for link buckets, resolved to the source bucket's
+   * layout. OBJECT_STORE layouts are cached like any other; the caller is
+   * responsible for validating the returned layout.

Review Comment:
   The inline Javadoc link is split across two lines (`{@link ClientProtocol` then `#getBucketDetails}`), which can render incorrectly and may trigger doclint/javadoc parsing warnings. Keep the `{@link ...}` reference on a single line (you can wrap the surrounding sentence without breaking the tag).



-- 
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: issues-unsubscribe@ozone.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org