hemantk-12 commented on code in PR #8124:
URL: https://github.com/apache/ozone/pull/8124#discussion_r2034058495
##########
hadoop-ozone/cli-shell/src/main/java/org/apache/hadoop/ozone/shell/snapshot/ListSnapshotDiffHandler.java:
##########
@@ -37,38 +38,32 @@ public class ListSnapshotDiffHandler extends Handler {
@CommandLine.Mixin
private BucketUri snapshotPath;
- @CommandLine.Option(names = {"-s", "--status"},
+ @CommandLine.Option(names = {"-js", "--job-status"},
Review Comment:
Removed `-js` option.
##########
hadoop-ozone/client/src/main/java/org/apache/hadoop/ozone/client/ObjectStore.java:
##########
@@ -731,19 +732,75 @@ public CancelSnapshotDiffResponse
cancelSnapshotDiff(String volumeName,
/**
* Get a list of the SnapshotDiff jobs for a bucket based on the JobStatus.
- * @param volumeName Name of the volume to which the snapshotted bucket
belong
- * @param bucketName Name of the bucket to which the snapshots belong
- * @param jobStatus JobStatus to be used to filter the snapshot diff jobs
- * @param listAll Option to specify whether to list all jobs or not
- * @return a list of SnapshotDiffJob objects
+ *
+ * @param volumeName Name of the volume to which the snapshotted
bucket belong
+ * @param bucketName Name of the bucket to which the snapshots
belong
+ * @param jobStatus JobStatus to be used to filter the snapshot
diff jobs
+ * @param listAll Option to specify whether to list all jobs or
not
+ * @param prevSnapshotDiffJob list snapshot diff jobs after this snapshot
diff job.
+ * @return an iterator of SnapshotDiffJob objects
* @throws IOException in case there is a failure while getting a response.
*/
- public List<OzoneSnapshotDiff> listSnapshotDiffJobs(String volumeName,
- String bucketName,
- String jobStatus,
- boolean listAll)
+ public Iterator<OzoneSnapshotDiff> listSnapshotDiffJobs(String volumeName,
+ String bucketName,
+ String jobStatus,
+ boolean listAll,
+ String
prevSnapshotDiffJob)
Review Comment:
Changed the signature to avoid it in the future.
##########
hadoop-ozone/client/src/main/java/org/apache/hadoop/ozone/client/ObjectStore.java:
##########
@@ -731,19 +732,75 @@ public CancelSnapshotDiffResponse
cancelSnapshotDiff(String volumeName,
/**
* Get a list of the SnapshotDiff jobs for a bucket based on the JobStatus.
- * @param volumeName Name of the volume to which the snapshotted bucket
belong
- * @param bucketName Name of the bucket to which the snapshots belong
- * @param jobStatus JobStatus to be used to filter the snapshot diff jobs
- * @param listAll Option to specify whether to list all jobs or not
- * @return a list of SnapshotDiffJob objects
+ *
+ * @param volumeName Name of the volume to which the snapshotted
bucket belong
+ * @param bucketName Name of the bucket to which the snapshots
belong
+ * @param jobStatus JobStatus to be used to filter the snapshot
diff jobs
+ * @param listAll Option to specify whether to list all jobs or
not
+ * @param prevSnapshotDiffJob list snapshot diff jobs after this snapshot
diff job.
Review Comment:
Reverted it. The auro-indent added it.
--
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
