Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11302 )
Change subject: util: add once class based on std::call_once ...................................................................... util: add once class based on std::call_once The existing KuduOnceDynamic class only allows calling functions of type Status(void). I intend on adding an argument to some of the existing the Status(void) methods that get called using a KuduOnceDynamic. As such, this patch adds a KuduOnceLambda class that accepts a lambda and otherwise has the same semantics as KuduOnceDynamic. I've replaced the KuduOnceDynamics that call methods that I intend on addding arguments to with their corresponding KuduOnceLambdas. Change-Id: Ide56057a800bf07923031df5a2a76a42f0b15358 Reviewed-on: http://gerrit.cloudera.org:8080/11302 Reviewed-by: Grant Henke <granthe...@apache.org> Tested-by: Andrew Wong <aw...@cloudera.com> Reviewed-by: Adar Dembo <a...@cloudera.com> --- M src/kudu/cfile/bloomfile.cc M src/kudu/cfile/bloomfile.h M src/kudu/cfile/cfile_reader.cc M src/kudu/cfile/cfile_reader.h M src/kudu/tablet/deltafile.cc M src/kudu/tablet/deltafile.h M src/kudu/util/once-test.cc M src/kudu/util/once.cc M src/kudu/util/once.h 9 files changed, 94 insertions(+), 30 deletions(-) Approvals: Grant Henke: Looks good to me, but someone else must approve Andrew Wong: Verified Adar Dembo: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/11302 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ide56057a800bf07923031df5a2a76a42f0b15358 Gerrit-Change-Number: 11302 Gerrit-PatchSet: 4 Gerrit-Owner: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Dan Burkert <danburk...@apache.org> Gerrit-Reviewer: Grant Henke <granthe...@apache.org> Gerrit-Reviewer: Kudu Jenkins