Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/23490 )
Change subject: KUDU-1261 instantiate templates for RowPtr::GetArray() ...................................................................... Patch Set 1: > Overall looks good. > Just one thing: there seems to be a build error in release mode. Thank you for the review! Indeed, I missed adding instantiations for DECIMAL32 and DECIMAL64 -- didn't notice that since I was using debug builds for testing and verification. I'm updating the patch shortly. -- To view, visit http://gerrit.cloudera.org:8080/23490 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I50b36c48959b00874921c6b56d210cf7358619f4 Gerrit-Change-Number: 23490 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Marton Greber <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]> Gerrit-Comment-Date: Fri, 03 Oct 2025 16:27:33 +0000 Gerrit-HasComments: No
