Alexey Serbin has submitted this change and it was merged. (
http://gerrit.cloudera.org:8080/22305 )
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
Reviewed-on: http://gerrit.cloudera.org:8080/22074
Tested-by: Kudu Jenkins
Reviewed-by: Yifan Zhang <[email protected]>
(cherry picked from commit 0c30c723e419f0a659cb7b809e3a11c539bf227b)
Reviewed-on: http://gerrit.cloudera.org:8080/22305
Reviewed-by: Abhishek Chennaka <[email protected]>
Tested-by: Alexey Serbin <[email protected]>
---
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(-)
Approvals:
Abhishek Chennaka: Looks good to me, approved
Alexey Serbin: Verified
--
To view, visit http://gerrit.cloudera.org:8080/22305
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: branch-1.18.x
Gerrit-MessageType: merged
Gerrit-Change-Id: I0a6f82f4da173949f2fc40517ebacd34be3a0434
Gerrit-Change-Number: 22305
Gerrit-PatchSet: 2
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]>
