Todd Lipcon has posted comments on this change. Change subject: [java] Reuse JVM across tests ......................................................................
Patch Set 1: Not sure how I feel about this change. In my experience these kinds of things usually cause more hard-to-diagnose bugs than they are worth -- eg random tests end up leaving threads running in failure scenarios, and those threads end up logging stuff captured by future tests, interfering with them, etc. If we want to improve test runtime I think we're better off focusing on distributed execution or parallel execution across many VMs, but still having a one-vm-per-test. -- To view, visit http://gerrit.cloudera.org:8080/7825 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8e4c6d53054d5da4f2ecf9f4130a51df13d4fa6c Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Grant Henke <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-HasComments: No
