Hello Tidy Bot, 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 (#4). 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 build-support/release/rat_exclude_files.txt M src/kudu/fs/block_manager-stress-test.cc M src/kudu/gutil/strings/split_internal.h M src/kudu/tablet/compaction_policy-test.cc M src/kudu/tools/CMakeLists.txt A src/kudu/tools/testdata/bad-diagnostics-log.txt A src/kudu/tools/testdata/sample-diagnostics-log.txt M src/kudu/tools/tool_action.h A src/kudu/tools/tool_action_diagnose.cc M src/kudu/tools/tool_main.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 15 files changed, 511 insertions(+), 34 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/00/9900/4 -- 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: 4 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>