Sailesh Mukil has uploaded a new patch set (#4). Change subject: IMPALA-5198: Error messages are sometimes dropped before reaching client ......................................................................
IMPALA-5198: Error messages are sometimes dropped before reaching client The Status::ToThrift() function takes the ErrorMsg, and pushes both the msg() and details() into the TStatus::error_msgs list. However, when we unpack the TStatus object into a Status object, we just copy all the TStatus::error_msgs to Status::ErrorMsg::details_ and leave Status::ErrorMsg::message_ blank. This led to the error message not being printed in certain cases. The beeswax server had code to comprimise for this setback, which is now removed. Change-Id: I5d9d63610eb0d2acae3a9303ce46e1410727ce87 --- M be/src/common/status.cc M be/src/common/status.h M be/src/service/impala-beeswax-server.cc M be/src/util/error-util.h 4 files changed, 30 insertions(+), 19 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/27/6627/4 -- To view, visit http://gerrit.cloudera.org:8080/6627 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I5d9d63610eb0d2acae3a9303ce46e1410727ce87 Gerrit-PatchSet: 4 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Sailesh Mukil <sail...@cloudera.com> Gerrit-Reviewer: Matthew Jacobs <m...@cloudera.com> Gerrit-Reviewer: Sailesh Mukil <sail...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>