cshannon commented on PR #3036: URL: https://github.com/apache/accumulo/pull/3036#issuecomment-1286978227
This is definitely a nice change to speed things up. I will just echo @ctubbsii here and say my main concern is just that the instance properties are modified in some tests and you don't know the order the tests will run so it's not really always testing the same thing potentially. Maybe when the integration test first starts up we capture the original properties/defaults for the instance and then after each test the original properties could be reset back so that when each test runs it always has the same starting point? -- 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]
