impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sailesh Mukil (Code Review)" <ger...@cloudera.org>
Subject [Impala-ASF-CR] IMPALA-5198: test exchange delays custom cluster test failed on s3 tests
Date Thu, 13 Apr 2017 17:33:18 GMT
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 <sailesh@cloudera.com>

Mime
View raw message