timmylicheng commented on issue #663: HDDS-3156 update allocateContainer to remove additional createPipeline step. URL: https://github.com/apache/hadoop-ozone/pull/663#issuecomment-602405476 > @timmylicheng thanks for the clarification. > > As we create the pipeline via heartbeat now it's not good to have it in the critical path; unless there are no pipelines available, we should try to avoid the creation of pipelines in allocate container calls and leave the job to background pipeline creator. > > I can see that changing all the related tests seems hectic and we should also have a discussion before making one node pipeline creation default in background pipeline creator. > > I'm good with the current changes for now. We can revisit this later. @nandakumar131 Sounds fair to me. Let's revisit the Ratis One pipeline later since it's not blocking anyone's use cases in real cluster.
---------------------------------------------------------------- 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] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
