Marton Greber has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/23494 )

Change subject: KUDU-1261 Python array datatype fixes
......................................................................


Patch Set 1:

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.


--
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: 1
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 08:22:28 +0000
Gerrit-HasComments: No

Reply via email to