JacksonYao287 commented on pull request #2172: URL: https://github.com/apache/ozone/pull/2172#issuecomment-829390297
@avijayanhwx @symious @GlenGeng please take a review. I rewrite this pr , there a two big changes : 1 add a new SCM service "getExistContainerWithPipelinesInBatch" , which will ask scm that which containers in the given list can be found by scm and will never throw an exception at client-side. "ReconContainerManager#checkAndAddNewContainerBatch" now use this method instead of "getContainerWithPipelineBatch" , which may throw an exception and drop the whole reports in a heartbeat. 2 fix a bug. for now , a closed container may be reported by datanode, and the pipeline of this container may be closed , recon can not sync this pipeline info from scm. so recon should have this ability and a no-open container to it's `containerStateMap` without the corresponding pipeling info. -- 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: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
