You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@impala.apache.org by "Bharath Vissapragada (Code Review)" <ge...@cloudera.org> on 2018/10/28 23:13:44 UTC

[Impala-ASF-CR] IMPALA-7727: Fix TStatusCode to TErrorCode mapping

Hello Michael Ho, Tim Armstrong, 

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/11778

to look at the new patch set (#2).

Change subject: IMPALA-7727: Fix TStatusCode to TErrorCode mapping
......................................................................

IMPALA-7727: Fix TStatusCode to TErrorCode mapping

- Uses a "GENERAL" TErrorCode type for all non-OK statuses.
- Detailed regression root cause description in the jira IMPALA-7727.
- Added a regression test.

Change-Id: Ie62527734aa73c1524c731773638590bdac9e789
---
M be/src/common/status.cc
M be/src/service/child-query.cc
M be/src/util/error-util.cc
M be/src/util/error-util.h
M testdata/workloads/functional-query/queries/QueryTest/compute-stats.test
5 files changed, 35 insertions(+), 24 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/78/11778/2
-- 
To view, visit http://gerrit.cloudera.org:8080/11778
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ie62527734aa73c1524c731773638590bdac9e789
Gerrit-Change-Number: 11778
Gerrit-PatchSet: 2
Gerrit-Owner: Bharath Vissapragada <bh...@cloudera.com>
Gerrit-Reviewer: Bharath Vissapragada <bh...@cloudera.com>
Gerrit-Reviewer: Michael Ho <kw...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <ta...@cloudera.com>