Hello Michael Ho, Thomas Tauber-Marshall, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/14677
to look at the new patch set (#8).
Change subject: IMPALA-9137: Blacklist node if a DataStreamService RPC to the
node fails
......................................................................
IMPALA-9137: Blacklist node if a DataStreamService RPC to the node fails
Introduces a new optional field to ReportExecStatusRequestPB:
StatusAuxInfoPB. StatusAuxInfoPB contains optional metadata associated
with ReportExecStatusRequestPB::overall_status. Currently,
StatusAuxInfoPB only contains one field: RPCErrorMessagePB, which is only
set if the ReportExecStatusRequestPB::fragment_instance_id failed
because a RPC to another impalad failed. The RPCErrorMessagePB contains
the destination node of the failed RPC.
Coordinator::UpdateBackendExecStatus(ReportExecStatusRequestPB, ...)
uses the information in RPCErrorMessagePB (if one is set) to blacklist
the target node.
Testing:
* Ran core tests
* Added new test to test_blacklist.py
Change-Id: I733cca13847fde43c8ea2ae574d3ae04bd06419c
---
M be/src/runtime/coordinator.cc
M be/src/runtime/coordinator.h
M be/src/runtime/krpc-data-stream-sender.cc
M be/src/runtime/query-state.cc
M be/src/runtime/runtime-state.h
A be/src/runtime/status-aux-info.h
M be/src/util/network-util.cc
M be/src/util/network-util.h
M common/protobuf/common.proto
M common/protobuf/control_service.proto
M tests/custom_cluster/test_blacklist.py
11 files changed, 189 insertions(+), 1 deletion(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/77/14677/8
--
To view, visit http://gerrit.cloudera.org:8080/14677
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I733cca13847fde43c8ea2ae574d3ae04bd06419c
Gerrit-Change-Number: 14677
Gerrit-PatchSet: 8
Gerrit-Owner: Sahil Takiar <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Michael Ho <[email protected]>
Gerrit-Reviewer: Sahil Takiar <[email protected]>
Gerrit-Reviewer: Thomas Tauber-Marshall <[email protected]>
