Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/22490 )
Change subject: [cbtree] modernize code in CBTree ...................................................................... [cbtree] modernize code in CBTree * introduce 'if constexpr' where appropriate * NULL --> nullptr * use static_cast<> instead of C-style cast * clean-up const-correctness a bit (needs a lot more) * COMPILE_ASSERT --> static_assert * CHECK/DCHECK cleanup * a few other style updates (haven't corrected everything) Change-Id: I4e6dbade4051dbf4934294dcd613a8c7deeb6277 Reviewed-on: http://gerrit.cloudera.org:8080/22485 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Yifan Zhang <[email protected]> (cherry picked from commit 08756b17aafcd7f42c6c867d951f4648f52ab083) Reviewed-on: http://gerrit.cloudera.org:8080/22490 --- M src/kudu/tablet/cbtree-test.cc M src/kudu/tablet/concurrent_btree.h 2 files changed, 41 insertions(+), 43 deletions(-) Approvals: Yifan Zhang: Looks good to me, approved Alexey Serbin: Verified -- To view, visit http://gerrit.cloudera.org:8080/22490 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.18.x Gerrit-MessageType: merged Gerrit-Change-Id: I4e6dbade4051dbf4934294dcd613a8c7deeb6277 Gerrit-Change-Number: 22490 Gerrit-PatchSet: 2 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: Yifan Zhang <[email protected]>
