swamirishi commented on code in PR #4678:
URL: https://github.com/apache/ozone/pull/4678#discussion_r1222365282
##########
hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/snapshot/SnapshotDiffManager.java:
##########
@@ -481,7 +479,7 @@ private SnapshotDiffReportOzone createPageResponse(
private void checkReportsIntegrity(final SnapshotDiffJob diffJob,
final int totalDiffEntries)
throws IOException {
- if (diffJob.getTotalDiffEntries() != totalDiffEntries) {
+ if (diffJob.getTotalDiffEntries() > totalDiffEntries) {
Review Comment:
Done. Here there was a bug which could mark the snapshot as failed if the
client passes the wrong token. I believe we don't validate the token's validity
anywhere. Made the changes to fix it. @hemantk-12 @smengcl please take a second
look at this if I am wrong.
--
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]
