Sailesh Mukil has uploaded a new change for review. http://gerrit.cloudera.org:8080/6627
Change subject: IMPALA-5198: test_exchange_delays custom cluster test failed on s3 tests ...................................................................... IMPALA-5198: test_exchange_delays custom cluster test failed on s3 tests 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, and is similar in behavior to the HS2 server. Change-Id: I5d9d63610eb0d2acae3a9303ce46e1410727ce87 --- M be/src/service/impala-beeswax-server.cc M be/src/util/error-util.cc M be/src/util/error-util.h 3 files changed, 13 insertions(+), 7 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/27/6627/1 -- To view, visit http://gerrit.cloudera.org:8080/6627 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I5d9d63610eb0d2acae3a9303ce46e1410727ce87 Gerrit-PatchSet: 1 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Sailesh Mukil <sail...@cloudera.com>
