rakeshadr commented on a change in pull request #2867:
URL: https://github.com/apache/ozone/pull/2867#discussion_r756554406



##########
File path: 
hadoop-ozone/client/src/main/java/org/apache/hadoop/ozone/client/rpc/RpcClient.java
##########
@@ -755,6 +756,20 @@ public OzoneBucket getBucketDetails(
     verifyBucketName(bucketName);
     OmBucketInfo bucketInfo =
         ozoneManagerClient.getBucketInfo(volumeName, bucketName);
+
+    BucketLayout bucketLayout = bucketInfo.getBucketLayout();

Review comment:
       @JyotinderSingh IMHO listing `#linkBucket ` implementation can be moved 
out to a separate jira. Apart from the list operation the patch looks good to 
me and those changes are enough to unblock the long waiting patch HDDS-5959, 
which has to be pushed in asap to unblock all the follow up tasks Please create 
a new sub-task to implement the listing of linkBucket content and rebase this 
patch to include only the other changes. Does that sound good?




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

Reply via email to