impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Dan Hecht (Code Review)" <ger...@cloudera.org>
Subject [Impala-ASF-CR] IMPALA-5853: fix GetResultSetMetadata() error message for invalid query id
Date Mon, 28 Aug 2017 22:34:22 GMT
Hello Henry Robinson,

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

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

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

Change subject: IMPALA-5853: fix GetResultSetMetadata() error message for invalid query id
......................................................................

IMPALA-5853: fix GetResultSetMetadata() error message for invalid query id

When given an invalid query id, most RPCs result with "Invalid query
id", but GetResultSetMetadata() response with "Unable to find session ID
for query handle".  That's confusing; make it consistent with other
RPCs.

Change-Id: I51eecf353f9cecaf88cecc9392b08ac1b9326b66
---
M be/src/service/impala-hs2-server.cc
M be/src/service/impala-server.cc
M be/src/service/impala-server.h
M tests/hs2/test_hs2.py
4 files changed, 14 insertions(+), 31 deletions(-)


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

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I51eecf353f9cecaf88cecc9392b08ac1b9326b66
Gerrit-PatchSet: 2
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Dan Hecht <dhecht@cloudera.com>
Gerrit-Reviewer: Henry Robinson <henry@cloudera.com>

Mime
View raw message