bshashikant commented on pull request #1717: URL: https://github.com/apache/ozone/pull/1717#issuecomment-756521522
> @bshashikant Thanks for working on the PR! The changes look good to me. Do we need to set the config `ScmConfigKeys.OZONE_DATANODE_PIPELINE_LIMIT` in tests? It should work by default? I have other minor comments inline. @lokeshj1703 , thanks for looking into this. I have added the limit on some specific tests and these tests need only one pipeline to function properly. Enabling multiRaft by default for these break the assumptions made in the tests, ---------------------------------------------------------------- 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]
