keith-turner opened a new pull request, #5676: URL: https://github.com/apache/accumulo/pull/5676
This changes does two things. First it fixes a problem where when running SimpleSuite test directly their directory name would include UnknownITClass instead of the actual class name. Second it gives each mini test directory a unique name. This helps debug issues when the accumulo build reruns a flaky test. Before this change when that happened the logs of the first run of the flaky test were lost. This behavior has prevented debugging of issues the the multi hour build finds on multiple occasions, wasting developer time and build time. After this change the dirs in `accumulo/test/target/mini-tests` look like the following after running `mvn clean verify -Psunny`. ``` org.apache.accumulo.suites.SimpleSharedMacTestSuiteIT-1750787155512-6711_SharedMiniClusterBase org.apache.accumulo.test.functional.ReadWriteIT-1750787009208-4495_interleaved org.apache.accumulo.test.functional.ReadWriteIT-1750787025424-3909_sunnyDay org.apache.accumulo.test.functional.ReadWriteIT-1750787058849-14476_largeTest org.apache.accumulo.test.functional.WALSunnyDayIT-1750787077639-32139_test ``` -- 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: notifications-unsubscr...@accumulo.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
