Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/23485 )
Change subject: KUDU-1261 Add array type support to Python client ...................................................................... Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/23485/1//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/23485/1//COMMIT_MSG@30 PS1, Line 30: - Fix: Empty BOOL array read triggers C++ assertion failure : (requires C++ client fix) > The current C++'s client's test coverage is spotty, indeed. I'll work on a Alright, this is fixed with http://gerrit.cloudera.org:8080/23490 If you are planning to revv this patch, maybe you could review the patch with the fix, push it if it look OK as-is, and re-base your patch on top of the fix. Thanks a lot! http://gerrit.cloudera.org:8080/#/c/23485/1/python/kudu/tests/test_array_datatype.py File python/kudu/tests/test_array_datatype.py: http://gerrit.cloudera.org:8080/#/c/23485/1/python/kudu/tests/test_array_datatype.py@259 PS1, Line 259: # Skip bool. : # TODO: empty BOOL arrays cause C++ Client assertion failure : if type_name == 'bool': : continue With http://gerrit.cloudera.org:8080/23490 it's no longer necessary to skip BOOL arrays verification. -- To view, visit http://gerrit.cloudera.org:8080/23485 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I2329c7466534bd4961860c05b600e7d4b4a11507 Gerrit-Change-Number: 23485 Gerrit-PatchSet: 1 Gerrit-Owner: Marton Greber <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Gabriella Lotz <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Zoltan Martonka <[email protected]> Gerrit-Comment-Date: Fri, 03 Oct 2025 07:45:27 +0000 Gerrit-HasComments: Yes
