Adar Dembo has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12031 )

Change subject: [test] Break up test output per test case.
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/12031/1/java/gradle/tests.gradle
File java/gradle/tests.gradle:

http://gerrit.cloudera.org:8080/#/c/12031/1/java/gradle/tests.gradle@32
PS1, Line 32:   // Break up the stdout and stderr logging by test case.
Don't you want to do it in the fancy closure style? Like

  reports {
    junitXml {
      outputPerTestCase(true)
    }
  }

Or some such.



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If7564e48cddc7bf7ea7595d925f605802259dced
Gerrit-Change-Number: 12031
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Will Berkeley <[email protected]>
Gerrit-Comment-Date: Tue, 04 Dec 2018 21:32:56 +0000
Gerrit-HasComments: Yes

Reply via email to