Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/18584
to look at the new patch set (#2).
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 a DCHECK that indicate the
compaction error occured.
Change-Id: Id257a1ce04fb2043d79641f770d03393ec5b844b
---
M src/kudu/integration-tests/write_limit-itest.cc
M src/kudu/tablet/rowset_info.cc
2 files changed, 5 insertions(+), 0 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/84/18584/2
--
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: newpatchset
Gerrit-Change-Id: Id257a1ce04fb2043d79641f770d03393ec5b844b
Gerrit-Change-Number: 18584
Gerrit-PatchSet: 2
Gerrit-Owner: Yifan Zhang <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
