Marton Greber has posted comments on this change. ( http://gerrit.cloudera.org:8080/23358 )
Change subject: Deflake testRandomBackupAndRestore ...................................................................... Patch Set 1: (1 comment) Yes the issue is not trivial at first due to the level of the stack trace. Anyway I've added the full stack traces for both errors. My reasoning was that the stack trace shows that we don't really hit issue in partial row. Moreover looking at the scala code backup and restore has its own big switch cases for partial row value getting and setting. I guess this re-implementation can have such side-effects. 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 Done -- 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: Marton Greber <[email protected]> Gerrit-Reviewer: Zoltan Chovan <[email protected]> Gerrit-Comment-Date: Fri, 12 Sep 2025 11:52:25 +0000 Gerrit-HasComments: Yes
