Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17035 )
Change subject: [gutil] update map-util to benefit from C++17 ...................................................................... [gutil] update map-util to benefit from C++17 This patch updates map-util to benefit from C++17 features such as try_emplace() and insert_or_assign() for dictionary containers: see [1], [2], [3], [4] for details. At least, with try_emplace() a new element isn't ever constructed if the container already contains an element with the specified key. [1] https://en.cppreference.com/w/cpp/container/map/try_emplace [2] https://en.cppreference.com/w/cpp/container/map/insert_or_assign [3] https://en.cppreference.com/w/cpp/container/unordered_map/try_emplace [4] https://en.cppreference.com/w/cpp/container/unordered_map/insert_or_assign Change-Id: I248c03e604855434ea3a0891c311461a455b428e Reviewed-on: http://gerrit.cloudera.org:8080/17035 Reviewed-by: Bankim Bhavsar <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/gutil/map-util.h 1 file changed, 35 insertions(+), 24 deletions(-) Approvals: Bankim Bhavsar: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/17035 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I248c03e604855434ea3a0891c311461a455b428e Gerrit-Change-Number: 17035 Gerrit-PatchSet: 4 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Bankim Bhavsar <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Mahesh Reddy <[email protected]>
