smengcl commented on pull request #1286: URL: https://github.com/apache/hadoop-ozone/pull/1286#issuecomment-670091470
> > @bharatviswa504 Now that we can prove the tests work with both ozone.om.enable.filesystem.paths options. Should we >proceed with [fdbc71c](https://github.com/apache/hadoop-ozone/commit/fdbc71cdabcfa846d538878f80bb602a37a63528) ? > > Can we use @BeforeClass for starting MiniOzoneCluster instead for each test, that would save time. > I would like to keep this parameterized, if anything is broken it would be easily caught, as default is false. (because of this, there were some cases missed) Good idea. Just have to make everything static. Let me try. ---------------------------------------------------------------- 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]
