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

Change subject: KUDU-1261 add array.fbs into common/serdes
......................................................................


Patch Set 3:

> Will this need followup changes to support decimal type?

I'm not planning to support DECIMAL128 in this iteration at all.  Same for 
INT128.

As for DECIMAL32 and DECIMAL64, they are covered by UInt32 and UInt64.  If 
necessary, we can add corresponding shims on top of existing union descriptors.


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie2c25aa32e3a72aada61e8980eb2613f8f649481
Gerrit-Change-Number: 23364
Gerrit-PatchSet: 3
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Wed, 03 Sep 2025 23:29:20 +0000
Gerrit-HasComments: No

Reply via email to