Mike Percy has submitted this change and it was merged. Change subject: tablet_copy-itest: Reduce flakiness ......................................................................
tablet_copy-itest: Reduce flakiness This patch reduces the flakiness under heavy CPU load of TabletCopyITest.TestDisableTabletCopy_NoTightLoopWhenTabletDeleted from 37/100 failures [1] to 0/100 failures [2] on dist-test by simply tuning up the # of log messages per second we tolerate from 30 to 60. This patch also fixes a minor bug where we attempt to send a tablet copy request with invalid parameters even if the PrepareTabletCopyRequest() method returns an error, indicating that it wasn't able to fill in the request. This resulted in warning messages in the logs that looked like the following: [libprotobuf ERROR /home/mpercy/src/kudu/thirdparty/src/protobuf-2.6.1/src/google/protobuf/message_lite.cc:123] Can't parse message of type "kudu.consensus.StartTabletCopyRequestPB" because it is missing required fields: tablet_id, copy_peer_uuid, copy_peer_addr W0428 19:46:21.691408 10560 service_if.cc:62] invalid parameter for call kudu.consensus.ConsensusService.StartTabletCopy: missing fields: tablet_id, copy_peer_uuid, copy_peer_addr [1] http://dist-test.cloudera.org/job?job_id=mpercy.1493408511.5800 [2] http://dist-test.cloudera.org/job?job_id=mpercy.1493408241.3403 Change-Id: Idd10f2fefb67634031f5c08e2adddc695193afb7 Reviewed-on: http://gerrit.cloudera.org:8080/6764 Reviewed-by: Adar Dembo <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/consensus/consensus_peers.cc M src/kudu/consensus/consensus_queue.cc M src/kudu/integration-tests/tablet_copy-itest.cc 3 files changed, 16 insertions(+), 15 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/6764 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idd10f2fefb67634031f5c08e2adddc695193afb7 Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Mike Percy <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]>
