Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/23565 )
Change subject: KUDU-1261 [Java] Add spark bindings for Array columns ...................................................................... Patch Set 12: Code-Review+2 (2 comments) http://gerrit.cloudera.org:8080/#/c/23565/12/java/kudu-backup-common/src/main/scala/org/apache/kudu/backup/TableMetadata.scala File java/kudu-backup-common/src/main/scala/org/apache/kudu/backup/TableMetadata.scala: http://gerrit.cloudera.org:8080/#/c/23565/12/java/kudu-backup-common/src/main/scala/org/apache/kudu/backup/TableMetadata.scala@272 PS12, Line 272: IllegalStateException nit: would IllegalArgumentException be a better choice? From where does the 'metadata' parameter come in here? http://gerrit.cloudera.org:8080/#/c/23565/12/java/kudu-backup-common/src/main/scala/org/apache/kudu/backup/TableMetadata.scala@277 PS12, Line 277: IllegalStateException nit: would UnsupportedOperationException be a better choice here? -- To view, visit http://gerrit.cloudera.org:8080/23565 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I786fdd1cbfbb67b4895b2e95b89addbc04341746 Gerrit-Change-Number: 23565 Gerrit-PatchSet: 12 Gerrit-Owner: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Fri, 24 Oct 2025 06:00:31 +0000 Gerrit-HasComments: Yes
