Hello Yifan Zhang, Kudu Jenkins, Abhishek Chennaka,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/22074
to look at the new patch set (#3).
Change subject: [fs] simplify FsErrorManager
......................................................................
[fs] simplify FsErrorManager
This patch updates the code in FsErrorManager to use
* std::array instead of std::unordered_map
* std::mutex instead of Mutex
Also, {Master,TabletServer}::ShutdownImpl() now unregister all the error
handlers that have been registered.
Change-Id: I0a6f82f4da173949f2fc40517ebacd34be3a0434
---
M src/kudu/fs/error_manager.cc
M src/kudu/fs/error_manager.h
M src/kudu/master/master.cc
M src/kudu/tserver/tablet_server.cc
4 files changed, 32 insertions(+), 25 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/74/22074/3
--
To view, visit http://gerrit.cloudera.org:8080/22074
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I0a6f82f4da173949f2fc40517ebacd34be3a0434
Gerrit-Change-Number: 22074
Gerrit-PatchSet: 3
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]>
