Hello Dan Burkert, Kudu Jenkins, Adar Dembo, Grant Henke,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/11302
to look at the new patch set (#3).
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
---
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(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/02/11302/3
--
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: newpatchset
Gerrit-Change-Id: Ide56057a800bf07923031df5a2a76a42f0b15358
Gerrit-Change-Number: 11302
Gerrit-PatchSet: 3
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
