Marton Greber has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/23701 )

Change subject: [java] Fix test result xml generation
......................................................................


Patch Set 2: Verified+1 Code-Review+1

(1 comment)

Just one license thing I've noticed, else LGTM.

unrelated test failures:
release: org.apache.kudu.spark.kudu.DefaultSourceTest
tsan: catalog_manager-test.0, auto_leader_rebalancer-test.0

Did take a look at some of the dist test output and indeed the log files are 
now actual logs not __ARCHIVE_TOO_BIG__ morever TestKuduBackup is around the 
10-30 MB which look okay. Thanks for looking into this!

http://gerrit.cloudera.org:8080/#/c/23701/2/java/kudu-backup/src/test/resources/logback-test.xml
File java/kudu-backup/src/test/resources/logback-test.xml:

http://gerrit.cloudera.org:8080/#/c/23701/2/java/kudu-backup/src/test/resources/logback-test.xml@1
PS2, Line 1: <?xml version="1.0" encoding="UTF-8"?>
missing license on top of a new file.



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib1f3353598ed24d296043e11a966e5ac7ca60e7e
Gerrit-Change-Number: 23701
Gerrit-PatchSet: 2
Gerrit-Owner: Zoltan Chovan <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Gabriella Lotz <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <[email protected]>
Gerrit-Reviewer: Zoltan Chovan <[email protected]>
Gerrit-Reviewer: Zoltan Martonka <[email protected]>
Gerrit-Comment-Date: Fri, 21 Nov 2025 15:08:46 +0000
Gerrit-HasComments: Yes

Reply via email to