[ https://issues.apache.org/jira/browse/HDFS-11045?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16198703#comment-16198703 ]
Ewan Higgs commented on HDFS-11045: ----------------------------------- [~dan...@cloudera.com], can you rebase this? {quote} Karthik Kambatla just pointed out an alternative I should have thought of a long time ago: don't actually run the scanner. It looks like I should be able to easily take the ReportCompiler out of context and test it on its own.{quote} Indeed, if you're trying to test the throttling sleep calculation then there should be a function that receives some times and computes a sleep value. Then you pump it will all sorts of values and verify that it calculated the correct amount of time to sleep. If you want to test the throttling mechanism then I suggest wrapping {{Thread.sleep}} so it can be mocked and then make sure it was called with the appropriate values. > TestDirectoryScanner#testThrottling fails: Throttle is too permissive > --------------------------------------------------------------------- > > Key: HDFS-11045 > URL: https://issues.apache.org/jira/browse/HDFS-11045 > Project: Hadoop HDFS > Issue Type: Bug > Components: datanode > Affects Versions: 3.0.0-alpha2 > Reporter: John Zhuge > Assignee: Daniel Templeton > Priority: Minor > Labels: flaky-test > Attachments: HDFS-11045.001.patch, HDFS-11045.002.patch, > HDFS-11045.003.patch, HDFS-11045.004.patch, HDFS-11045.005.patch, > HDFS-11045.006.patch, HDFS-11045.007.patch, HDFS-11045.008.patch, > HDFS-11045.009.patch > > > TestDirectoryScanner.testThrottling:709 Throttle is too permissive > https://builds.apache.org/job/PreCommit-HDFS-Build/17259/artifact/patchprocess/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt -- This message was sent by Atlassian JIRA (v6.4.14#64029) --------------------------------------------------------------------- To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org