Hello Kudu Jenkins, Adar Dembo,

I'd like you to reexamine a change. Please visit

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

to look at the new patch set (#3).

Change subject: Fix flaky file_cache-test
......................................................................

Fix flaky file_cache-test

This test became flaky after 4db748f3021711a67017fe51f30097350f343101
since that commit added a chance that a background thread opened a file
descriptor during the run of this test, which makes assertions on file
descriptor count.

This patch fixes the issue by making the test only count fds which point
to files within the test dir, rather than all open fds.

Change-Id: I5bd51c267fbc7dc954d593caedf8631f261909c9
---
M src/kudu/fs/block_manager-stress-test.cc
M src/kudu/util/file_cache-stress-test.cc
M src/kudu/util/file_cache-test-util.h
M src/kudu/util/file_cache-test.cc
M src/kudu/util/test_util.cc
M src/kudu/util/test_util.h
6 files changed, 85 insertions(+), 26 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/00/9900/3
--
To view, visit http://gerrit.cloudera.org:8080/9900
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I5bd51c267fbc7dc954d593caedf8631f261909c9
Gerrit-Change-Number: 9900
Gerrit-PatchSet: 3
Gerrit-Owner: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>

Reply via email to