Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/23483 )

Change subject: KUDU-1261 [Java] Add read support for Array Type
......................................................................


Patch Set 20:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/23483/20//COMMIT_MSG
Commit Message:

PS20:
I think it's necessary to re-base this patch at least on top of 
https://github.com/apache/kudu/commit/ac505c986987e9079acf9cbd74d3ef56f728a94d

That's the patch where the provisions for empty/null validity bitset have 
appeared.  Without them, there might be CHECK/DCHEK or just nullptr 
dereferencing with SIGSEGV because the prior/outdated conventions on the 
validity vector.



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie26750cf540b0097c77e5454c1c1d20b3a194c52
Gerrit-Change-Number: 23483
Gerrit-PatchSet: 20
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 07:08:48 +0000
Gerrit-HasComments: Yes

Reply via email to