bharatviswa504 commented on a change in pull request #1174: HDDS-1856. Make required changes for Non-HA to use new HA code in OM. URL: https://github.com/apache/hadoop/pull/1174#discussion_r308989717
########## File path: hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/ratis/OzoneManagerDoubleBuffer.java ########## @@ -248,10 +291,20 @@ public long getFlushIterations() { * @param response * @param transactionIndex */ - public synchronized void add(OMClientResponse response, + public synchronized CompletableFuture<Void> add(OMClientResponse response, Review comment: This is for a temporary thing. Sooner, all OM uses HA, so this code will be removed in later point of time. And in HA case, we don't even use future. So, I think this should be okay, let me know you still want to use Optional here? ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org