Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/23482 )
Change subject: KUDU-1261 [Java] Add write support for Array Type ...................................................................... Patch Set 16: (1 comment) http://gerrit.cloudera.org:8080/#/c/23482/16/java/kudu-client/src/main/java/org/apache/kudu/client/PartialRow.java File java/kudu-client/src/main/java/org/apache/kudu/client/PartialRow.java: http://gerrit.cloudera.org:8080/#/c/23482/16/java/kudu-client/src/main/java/org/apache/kudu/client/PartialRow.java@992 PS16, Line 992: // Case 3: validity == null → infer validity from non-nulls : boolean[] inferred = new boolean[values.length]; : for (int i = 0; i < values.length; i++) { : inferred[i] = (values[i] != null); : } : return inferred; > I don't quite understand why the logic here is different from in the 'if (v Ah, I guess you want to have this behavior as a convenience for String and maybe boxed types, right? Then it would be great to document this behavior for the group of public methods below, otherwise it looks a bit off. Thanks! -- To view, visit http://gerrit.cloudera.org:8080/23482 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Icbe5e243eafe12a8977d40204dacab99624451eb Gerrit-Change-Number: 23482 Gerrit-PatchSet: 16 Gerrit-Owner: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Zoltan Chovan <[email protected]> Gerrit-Comment-Date: Thu, 23 Oct 2025 00:33:40 +0000 Gerrit-HasComments: Yes
