Adar Dembo has posted comments on this change.

Change subject: [java] Gradle test task improvements
......................................................................


Patch Set 4: Code-Review+2

(1 comment)

http://gerrit.cloudera.org:8080/#/c/7782/1//COMMIT_MSG
Commit Message:

PS1, Line 11: 
            : Change-Id: Ifb660b15423b68c299518b7ba5
> The root pom.xml sets reuseForks=false which means each test class executes
Ah, I missed the reuseForks=false thing in pom.xml. Carry on then.

But yes, in general I think we should try to restore reuseForks=true since it 
should also yield faster running tests. Out of scope for this patch, obviously.


-- 
To view, visit http://gerrit.cloudera.org:8080/7782
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ifb660b15423b68c299518b7ba54a15135972a007
Gerrit-PatchSet: 4
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Grant Henke <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-HasComments: Yes

Reply via email to