bharatviswa504 commented on a change in pull request #1631: HDDS-2188 : Implement LocatedFileStatus & getFileBlockLocations to pr… URL: https://github.com/apache/hadoop/pull/1631#discussion_r333269722
########## File path: hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/KeyManagerImpl.java ########## @@ -688,6 +663,48 @@ public OmKeyInfo lookupKey(OmKeyArgs args, String clientAddress) } } + /** + * Refresh pipeline info in OM by asking SCM. + * @param value OmKeyInfo + */ + @VisibleForTesting + protected void refreshPipeline(OmKeyInfo value) { + // Refresh container pipeline info from SCM + // based on OmKeyArgs.refreshPipeline flag + // 1. Client send initial read request OmKeyArgs.refreshPipeline = false Review comment: Currently as said in code comments, it is not happening. Initially itself it is called with refreshPipeline to true. ---------------------------------------------------------------- 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