SaketaChalamchala commented on code in PR #7988: URL: https://github.com/apache/ozone/pull/7988#discussion_r2011238181
########## hadoop-ozone/client/src/main/java/org/apache/hadoop/ozone/client/rpc/RpcClient.java: ########## @@ -1020,6 +1020,30 @@ public OzoneSnapshot getSnapshotInfo(String volumeName, return OzoneSnapshot.fromSnapshotInfo(snapshotInfo); } + /** + * {@inheritDoc} + */ + @Override + public OzoneSnapshot getSnapshotInfo(String volumeName, + String bucketName, + String snapshotName, + String omNodeId) throws IOException { Review Comment: Thanks for the patch @ivandika3. Can we have a single snapshot function definition and have clients set null omNodeIds if `--om-node-id` is not specified? -- 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-unsubscr...@ozone.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@ozone.apache.org For additional commands, e-mail: issues-h...@ozone.apache.org