Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/23494 )
Change subject: KUDU-1261 Python array datatype fixes ...................................................................... Patch Set 2: > Thanks for the feedback Alexey! > I'm fine with the current design. From Python's point of view my > previous concern was bogus, and in the Python client we have this > nice api that determines what to use int32 or 64 under the hood. I > also like the performance thought of yours. > If I'm not mistaken there will be the possibility to add 128 bit > version later (no breaking api right, IIUC). In that case we can > just see what feedback we get, how things unfold. Right, we can add 128 bit getters/setters later, make it work similar to what we have in scalar (non-array) case, and still keep the API (and ABI) for the Kudu C++ client compatible. Yes, I agree we can see how it goes, and process the feedback accordingly. Thanks! -- To view, visit http://gerrit.cloudera.org:8080/23494 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I77292b07a854b4f48a75d4e1293f302e8bad9129 Gerrit-Change-Number: 23494 Gerrit-PatchSet: 2 Gerrit-Owner: Marton Greber <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Gabriella Lotz <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Marton Greber <[email protected]> Gerrit-Comment-Date: Tue, 07 Oct 2025 13:44:06 +0000 Gerrit-HasComments: No
