duongkame commented on PR #4154: URL: https://github.com/apache/ozone/pull/4154#issuecomment-1374394849
> > * We should print a WARN message when it happens; see below. > * Also, let's keep the `addContainerToPipelineSCMStart` method for now so that it is easier to back port this change. We may do the code refactoring later. Thanks for the suggestions, @szetszwo. I've made the updates. Once this is merged I'll submit another PR to clean up the code. -- 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]
