You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@impala.apache.org by lv...@apache.org on 2018/03/05 03:34:42 UTC

[2/2] impala git commit: IMPALA-6338: xfail test_profile_fragment_instances

IMPALA-6338: xfail test_profile_fragment_instances

test_profile_fragment_instances has been flaky. The reason is
understood and low impact - there is a very hard to hit race condition
where queries that complete successfully but are cancelled internally
by the coordinator may not have a complete runtime profile.

A patch to fix this was already submitted, but after merging some
crashes were observed in tests so it was reverted. While we figure out
what is going on there, temporarily xfail the test.

Change-Id: I06adc9d8f412fa54b07bd1ebc2c062dd3adde1f7
Reviewed-on: http://gerrit.cloudera.org:8080/9467
Reviewed-by: Sailesh Mukil <sa...@cloudera.com>
Reviewed-by: David Knupp <dk...@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/8d450f65
Tree: http://git-wip-us.apache.org/repos/asf/impala/tree/8d450f65
Diff: http://git-wip-us.apache.org/repos/asf/impala/diff/8d450f65

Branch: refs/heads/2.x
Commit: 8d450f65704f2ce5804eace4d0fcec053388dff4
Parents: e5689fb
Author: Thomas Tauber-Marshall <tm...@cloudera.com>
Authored: Fri Mar 2 13:34:31 2018 -0800
Committer: Impala Public Jenkins <im...@gerrit.cloudera.org>
Committed: Sat Mar 3 22:34:01 2018 +0000

----------------------------------------------------------------------
 tests/query_test/test_observability.py | 1 +
 1 file changed, 1 insertion(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/impala/blob/8d450f65/tests/query_test/test_observability.py
----------------------------------------------------------------------
diff --git a/tests/query_test/test_observability.py b/tests/query_test/test_observability.py
index 58793af..e24cd44 100644
--- a/tests/query_test/test_observability.py
+++ b/tests/query_test/test_observability.py
@@ -117,6 +117,7 @@ class TestObservability(ImpalaTestSuite):
         in runtime_profile
 
   @SkipIfLocal.multiple_impalad
+  @pytest.mark.xfail(reason="IMPALA-6338")
   def test_profile_fragment_instances(self):
     """IMPALA-6081: Test that the expected number of fragment instances and their exec
     nodes appear in the runtime profile, even when fragments may be quickly cancelled when