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

Change subject: KUDU-1261 [Java] Implement serdes of Array Type column
......................................................................


Patch Set 8:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/23481/8/java/kudu-client/src/main/java/org/apache/kudu/client/Array1dSerdes.java
File java/kudu-client/src/main/java/org/apache/kudu/client/Array1dSerdes.java:

http://gerrit.cloudera.org:8080/#/c/23481/8/java/kudu-client/src/main/java/org/apache/kudu/client/Array1dSerdes.java@134
PS8, Line 134: Unexpected union type
I may be misunderstanding something, but shouldn't this be an unexpected data 
type?


http://gerrit.cloudera.org:8080/#/c/23481/8/java/kudu-client/src/main/java/org/apache/kudu/client/Array1dSerdes.java@192
PS8, Line 192: // FIX: return int[]
is this a todo for later?



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie1ad9f65fe94c8662ed0e0834ce849e078fc72d2
Gerrit-Change-Number: 23481
Gerrit-PatchSet: 8
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, 02 Oct 2025 09:00:52 +0000
Gerrit-HasComments: Yes

Reply via email to