Todd Lipcon has submitted this change and it was merged. Change subject: Remove gutil/algorithm.h ......................................................................
Remove gutil/algorithm.h This was included in a few places, but apparently wasn't used. The motivation to remove it now is twofold: a) we're on C++11 and most of these functions are implemented in <algorithm> now b) for the ones that aren't in C++11, apparently they aren't that generally useful for us if we've gone several years without using them. Change-Id: Ic4703a4b101bec940f9bb01b406aebdad70e1505 Reviewed-on: http://gerrit.cloudera.org:8080/2468 Tested-by: Kudu Jenkins Reviewed-by: Dan Burkert <[email protected]> --- M src/kudu/consensus/mt-log-test.cc D src/kudu/gutil/algorithm.h M src/kudu/gutil/stl_util.h M src/kudu/tablet/cfile_set.cc M src/kudu/tablet/delta_compaction-test.cc M src/kudu/tablet/deltafile-test.cc M src/kudu/tablet/diskrowset-test.cc M src/kudu/tablet/major_delta_compaction-test.cc M src/kudu/tablet/rowset_info.cc 9 files changed, 0 insertions(+), 449 deletions(-) Approvals: Dan Burkert: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/2468 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic4703a4b101bec940f9bb01b406aebdad70e1505 Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]>
