You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@impala.apache.org by mi...@apache.org on 2018/02/15 16:33:35 UTC
impala git commit: IMPALA-6512: Fix test_exchange_delays for KRPC
Repository: impala
Updated Branches:
refs/heads/master 19ab465b3 -> 9d3702192
IMPALA-6512: Fix test_exchange_delays for KRPC
The sender timed out error message diverges between Thrift and KRPC
slightly due to the source address being not readily available with
Thrift RPC implementation. This leads to failure in test_exchange_delays
when KRPC is enabled.
This change fixes the problem by shortening the error message string
to match against.
Testing done: Tested with KRPC enabled in the code and verified the tests passed.
Change-Id: Idd9410381dbb931231c92f084917265e5067b4c9
Reviewed-on: http://gerrit.cloudera.org:8080/9331
Reviewed-by: Sailesh Mukil <sa...@cloudera.com>
Reviewed-by: Tim Armstrong <ta...@cloudera.com>
Tested-by: Impala Public Jenkins
Project: http://git-wip-us.apache.org/repos/asf/impala/repo
Commit: http://git-wip-us.apache.org/repos/asf/impala/commit/9d370219
Tree: http://git-wip-us.apache.org/repos/asf/impala/tree/9d370219
Diff: http://git-wip-us.apache.org/repos/asf/impala/diff/9d370219
Branch: refs/heads/master
Commit: 9d370219273e5068ec447168516dd539985d9903
Parents: 19ab465
Author: Michael Ho <kw...@cloudera.com>
Authored: Wed Feb 14 16:54:39 2018 -0800
Committer: Impala Public Jenkins <im...@gerrit.cloudera.org>
Committed: Thu Feb 15 10:55:55 2018 +0000
----------------------------------------------------------------------
.../queries/QueryTest/exchange-delays-zero-rows.test | 2 +-
.../functional-query/queries/QueryTest/exchange-delays.test | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/impala/blob/9d370219/testdata/workloads/functional-query/queries/QueryTest/exchange-delays-zero-rows.test
----------------------------------------------------------------------
diff --git a/testdata/workloads/functional-query/queries/QueryTest/exchange-delays-zero-rows.test b/testdata/workloads/functional-query/queries/QueryTest/exchange-delays-zero-rows.test
index 79b7db4..9c8a096 100644
--- a/testdata/workloads/functional-query/queries/QueryTest/exchange-delays-zero-rows.test
+++ b/testdata/workloads/functional-query/queries/QueryTest/exchange-delays-zero-rows.test
@@ -7,5 +7,5 @@ where l_linenumber = -1
group by l_orderkey
---- RESULTS
---- CATCH
-Sender timed out waiting for receiver fragment instance
+timed out waiting for receiver fragment instance
====
http://git-wip-us.apache.org/repos/asf/impala/blob/9d370219/testdata/workloads/functional-query/queries/QueryTest/exchange-delays.test
----------------------------------------------------------------------
diff --git a/testdata/workloads/functional-query/queries/QueryTest/exchange-delays.test b/testdata/workloads/functional-query/queries/QueryTest/exchange-delays.test
index 0dac1d9..716085a 100644
--- a/testdata/workloads/functional-query/queries/QueryTest/exchange-delays.test
+++ b/testdata/workloads/functional-query/queries/QueryTest/exchange-delays.test
@@ -6,5 +6,5 @@ from tpch.lineitem
---- RESULTS
6001215
---- CATCH
-Sender timed out waiting for receiver fragment instance
+timed out waiting for receiver fragment instance
====