Sahil Takiar has uploaded a new patch set (#9). ( http://gerrit.cloudera.org:8080/14677 )
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/9 -- 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: 9 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]>
