Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9900 )
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 Reviewed-on: http://gerrit.cloudera.org:8080/9900 Reviewed-by: Adar Dembo <a...@cloudera.com> Tested-by: Kudu Jenkins --- 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, 84 insertions(+), 25 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- 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: merged Gerrit-Change-Id: I5bd51c267fbc7dc954d593caedf8631f261909c9 Gerrit-Change-Number: 9900 Gerrit-PatchSet: 6 Gerrit-Owner: Todd Lipcon <t...@apache.org> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <t...@apache.org>