smengcl commented on code in PR #4950:
URL: https://github.com/apache/ozone/pull/4950#discussion_r1238781253


##########
hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/OmSnapshotManager.java:
##########
@@ -885,4 +886,27 @@ public void close() {
   public long getDiffCleanupServiceInterval() {
     return diffCleanupServiceInterval;
   }
+
+  // Wait for double buffer to be flushed and confirm that the
+  // snapshot directory exists.
+  public void waitForSnapshotFlush(String volumeName, String bucketName,

Review Comment:
   Looks like this is only used in tests. We can add an annotation for that.
   
   ```suggestion
     @VisibleForTesting
     public void waitForSnapshotFlush(String volumeName, String bucketName,
   ```



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