Zoltan Chovan has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/23358 )

Change subject: Deflake testRandomBackupAndRestore
......................................................................


Patch Set 1:

(1 comment)

Nice catch Marton! I'm not sure I fully understand the failure scenario so I'll 
play with this a bit more locally, but at a first glance I wonder if this is 
actually a problem in PartialRow's getBinary() method?

http://gerrit.cloudera.org:8080/#/c/23358/1/java/kudu-backup/src/test/scala/org/apache/kudu/backup/TestKuduBackup.scala
File 
java/kudu-backup/src/test/scala/org/apache/kudu/backup/TestKuduBackup.scala:

http://gerrit.cloudera.org:8080/#/c/23358/1/java/kudu-backup/src/test/scala/org/apache/kudu/backup/TestKuduBackup.scala@342
PS1, Line 342:     // Before the fix, this would fail with: 
java.lang.NoClassDefFoundError: org/apache/commons/net/util/Base64
here and below, could you reformat the comments a bit so they fit into the 
rowlength char limit?



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ibbf05b23cb6c922ee880f4f1dda262be5e97452b
Gerrit-Change-Number: 23358
Gerrit-PatchSet: 1
Gerrit-Owner: Marton Greber <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Gabriella Lotz <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Zoltan Chovan <[email protected]>
Gerrit-Comment-Date: Thu, 28 Aug 2025 12:42:14 +0000
Gerrit-HasComments: Yes

Reply via email to