Yifan Zhang has uploaded this change for review. ( http://gerrit.cloudera.org:8080/18584
Change subject: KUDU-3354: Deflake DisableWriteWhenExceedingQuotaTest ...................................................................... KUDU-3354: Deflake DisableWriteWhenExceedingQuotaTest I noticed that the TestDisableWritePrivilegeWhenExceedingSizeQuota test failed some times in latest code review items because of test timeout. From the test log we can see maintenance manager tends to schedule a high score compact operation of some rowsets but do nothing in the end because 'Compaction resulted in no output rows (all input rows were GCed!)'. This leads to failures to schedule flush operations and to hit the size limit until run out of time. I think this weird high perf score most likely has something to do with not explicitly initializing the 'value_' of RowsetInfo after some debugging. The 'value_' can't be updated in FinalizeCDFVector() either, because in the test scenario it has nearly one key in a rowset, and the 'total_width' of the rowset is 0. I run the test 10 times and see no failure with this fix, compared to 5 failures without the fix but with the DCHECK in this patch. Change-Id: Id257a1ce04fb2043d79641f770d03393ec5b844b --- M src/kudu/integration-tests/write_limit-itest.cc M src/kudu/tablet/rowset_info.cc M src/kudu/tablet/tablet.cc 3 files changed, 6 insertions(+), 0 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/84/18584/1 -- To view, visit http://gerrit.cloudera.org:8080/18584 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Id257a1ce04fb2043d79641f770d03393ec5b844b Gerrit-Change-Number: 18584 Gerrit-PatchSet: 1 Gerrit-Owner: Yifan Zhang <[email protected]>
