[Impala-ASF-CR] IMPALA-6338: xfail test profile fragment instances

[Impala Public Jenkins (Code Review)]

Impala Public Jenkins has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/9467 )

Change subject: 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 
Reviewed-by: David Knupp 
Tested-by: Impala Public Jenkins
---
M tests/query_test/test_observability.py
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  Sailesh Mukil: Looks good to me, but someone else must approve
  David Knupp: Looks good to me, approved
  Impala Public Jenkins: Verified

--
To view, visit http://gerrit.cloudera.org:8080/9467
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I06adc9d8f412fa54b07bd1ebc2c062dd3adde1f7
Gerrit-Change-Number: 9467
Gerrit-PatchSet: 2
Gerrit-Owner: Thomas Tauber-Marshall 
Gerrit-Reviewer: David Knupp 
Gerrit-Reviewer: Impala Public Jenkins
Gerrit-Reviewer: Sailesh Mukil 

[Impala-ASF-CR] IMPALA-6338: xfail test profile fragment instances

[Impala Public Jenkins (Code Review)]

Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9467 )

Change subject: IMPALA-6338: xfail test_profile_fragment_instances
..


Patch Set 1: Verified+1


--
To view, visit http://gerrit.cloudera.org:8080/9467
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I06adc9d8f412fa54b07bd1ebc2c062dd3adde1f7
Gerrit-Change-Number: 9467
Gerrit-PatchSet: 1
Gerrit-Owner: Thomas Tauber-Marshall 
Gerrit-Reviewer: David Knupp 
Gerrit-Reviewer: Impala Public Jenkins
Gerrit-Reviewer: Sailesh Mukil 
Gerrit-Comment-Date: Sat, 03 Mar 2018 01:51:02 +
Gerrit-HasComments: No

[Impala-ASF-CR] IMPALA-6338: xfail test profile fragment instances

[Impala Public Jenkins (Code Review)]

Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9467 )

Change subject: IMPALA-6338: xfail test_profile_fragment_instances
..


Patch Set 1:

Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/2037/


--
To view, visit http://gerrit.cloudera.org:8080/9467
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I06adc9d8f412fa54b07bd1ebc2c062dd3adde1f7
Gerrit-Change-Number: 9467
Gerrit-PatchSet: 1
Gerrit-Owner: Thomas Tauber-Marshall 
Gerrit-Reviewer: David Knupp 
Gerrit-Reviewer: Impala Public Jenkins
Gerrit-Reviewer: Sailesh Mukil 
Gerrit-Comment-Date: Fri, 02 Mar 2018 21:58:52 +
Gerrit-HasComments: No

[Impala-ASF-CR] IMPALA-6338: xfail test profile fragment instances

[David Knupp (Code Review)]

David Knupp has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9467 )

Change subject: IMPALA-6338: xfail test_profile_fragment_instances
..


Patch Set 1: Code-Review+2


--
To view, visit http://gerrit.cloudera.org:8080/9467
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I06adc9d8f412fa54b07bd1ebc2c062dd3adde1f7
Gerrit-Change-Number: 9467
Gerrit-PatchSet: 1
Gerrit-Owner: Thomas Tauber-Marshall 
Gerrit-Reviewer: David Knupp 
Gerrit-Reviewer: Sailesh Mukil 
Gerrit-Comment-Date: Fri, 02 Mar 2018 21:58:21 +
Gerrit-HasComments: No

[Impala-ASF-CR] IMPALA-6338: xfail test profile fragment instances

[Sailesh Mukil (Code Review)]

Sailesh Mukil has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9467 )

Change subject: IMPALA-6338: xfail test_profile_fragment_instances
..


Patch Set 1: Code-Review+1


--
To view, visit http://gerrit.cloudera.org:8080/9467
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I06adc9d8f412fa54b07bd1ebc2c062dd3adde1f7
Gerrit-Change-Number: 9467
Gerrit-PatchSet: 1
Gerrit-Owner: Thomas Tauber-Marshall 
Gerrit-Reviewer: Sailesh Mukil 
Gerrit-Comment-Date: Fri, 02 Mar 2018 21:47:21 +
Gerrit-HasComments: No

[Impala-ASF-CR] IMPALA-6338: xfail test profile fragment instances

[Thomas Tauber-Marshall (Code Review)]

Thomas Tauber-Marshall has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/9467


Change subject: 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
---
M tests/query_test/test_observability.py
1 file changed, 1 insertion(+), 0 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/67/9467/1
--
To view, visit http://gerrit.cloudera.org:8080/9467
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I06adc9d8f412fa54b07bd1ebc2c062dd3adde1f7
Gerrit-Change-Number: 9467
Gerrit-PatchSet: 1
Gerrit-Owner: Thomas Tauber-Marshall