[kudu-CR](branch-1.2.x) [client-test] Reduce flakyness of TestWriteWithDeadTabletServer

[Jean-Daniel Cryans (Code Review)]

Jean-Daniel Cryans has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/8429 )

Change subject: [client-test] Reduce flakyness of TestWriteWithDeadTabletServer
..

[client-test] Reduce flakyness of TestWriteWithDeadTabletServer

This test fails often due to an unexpected reason for the write
timeout. This is related to KUDU-1466.

I just commented out the assertion and left a TODO to reenable
once we address the jira. Also added some info to the jira
itself.

Change-Id: Ice347a30c71b1cb3937cd1b30b0744fcd2dae0cf
Reviewed-on: http://gerrit.cloudera.org:8080/6198
Tested-by: Kudu Jenkins
Reviewed-by: Todd Lipcon 
(cherry picked from commit 02ce2a8a23fb127697c10dffea84dcfa2f2b3933)
Reviewed-on: http://gerrit.cloudera.org:8080/8429
Reviewed-by: Jean-Daniel Cryans 
---
M src/kudu/client/client-test.cc
1 file changed, 4 insertions(+), 1 deletion(-)

Approvals:
  Kudu Jenkins: Verified
  Todd Lipcon: Looks good to me, approved
  Jean-Daniel Cryans: Looks good to me, approved

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

Gerrit-Project: kudu
Gerrit-Branch: branch-1.2.x
Gerrit-MessageType: merged
Gerrit-Change-Id: Ice347a30c71b1cb3937cd1b30b0744fcd2dae0cf
Gerrit-Change-Number: 8429
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin 
Gerrit-Reviewer: David Ribeiro Alves 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon 
Gerrit-Reviewer: Will Berkeley 

[kudu-CR](branch-1.2.x) [client-test] Reduce flakyness of TestWriteWithDeadTabletServer

[Jean-Daniel Cryans (Code Review)]

Jean-Daniel Cryans has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/8429 )

Change subject: [client-test] Reduce flakyness of TestWriteWithDeadTabletServer
..


Patch Set 1: Code-Review+2


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

Gerrit-Project: kudu
Gerrit-Branch: branch-1.2.x
Gerrit-MessageType: comment
Gerrit-Change-Id: Ice347a30c71b1cb3937cd1b30b0744fcd2dae0cf
Gerrit-Change-Number: 8429
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin 
Gerrit-Reviewer: David Ribeiro Alves 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon 
Gerrit-Reviewer: Will Berkeley 
Gerrit-Comment-Date: Wed, 01 Nov 2017 22:22:32 +
Gerrit-HasComments: No

[kudu-CR](branch-1.2.x) [client-test] Reduce flakyness of TestWriteWithDeadTabletServer

[Todd Lipcon (Code Review)]

Todd Lipcon has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/8429 )

Change subject: [client-test] Reduce flakyness of TestWriteWithDeadTabletServer
..


Patch Set 1: Code-Review+2


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

Gerrit-Project: kudu
Gerrit-Branch: branch-1.2.x
Gerrit-MessageType: comment
Gerrit-Change-Id: Ice347a30c71b1cb3937cd1b30b0744fcd2dae0cf
Gerrit-Change-Number: 8429
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin 
Gerrit-Reviewer: David Ribeiro Alves 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon 
Gerrit-Reviewer: Will Berkeley 
Gerrit-Comment-Date: Wed, 01 Nov 2017 01:26:46 +
Gerrit-HasComments: No

[kudu-CR](branch-1.2.x) [client-test] Reduce flakyness of TestWriteWithDeadTabletServer

[Alexey Serbin (Code Review)]

Hello Kudu Jenkins, Todd Lipcon,

I'd like you to do a code review. Please visit

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

to review the following change.


Change subject: [client-test] Reduce flakyness of TestWriteWithDeadTabletServer
..

[client-test] Reduce flakyness of TestWriteWithDeadTabletServer

This test fails often due to an unexpected reason for the write
timeout. This is related to KUDU-1466.

I just commented out the assertion and left a TODO to reenable
once we address the jira. Also added some info to the jira
itself.

Change-Id: Ice347a30c71b1cb3937cd1b30b0744fcd2dae0cf
Reviewed-on: http://gerrit.cloudera.org:8080/6198
Tested-by: Kudu Jenkins
Reviewed-by: Todd Lipcon 
(cherry picked from commit 02ce2a8a23fb127697c10dffea84dcfa2f2b3933)
---
M src/kudu/client/client-test.cc
1 file changed, 4 insertions(+), 1 deletion(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/29/8429/1
--
To view, visit http://gerrit.cloudera.org:8080/8429
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.2.x
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ice347a30c71b1cb3937cd1b30b0744fcd2dae0cf
Gerrit-Change-Number: 8429
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin 
Gerrit-Reviewer: David Ribeiro Alves 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon