[
https://issues.apache.org/jira/browse/HDFS-10641?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15398265#comment-15398265
]
Mingliang Liu commented on HDFS-10641:
--------------------------------------
Thanks for the patch, [~daryn].
The {{blockingOp}} is a future task whose internal state is updated sometime
after {{runBlockOp}} returns. The q length is an internal state of the bm. Once
{{runBlockOp}} returns, {{blockingOp}} state is updated independently on the
bm. This is indeed not a problem of the wait condition. The v0 patch was to
wait for a longer time as {{blockingOp}} running state will be updated
eventually.
I think the v1 patch is good as well since the assertion is depending on a
dedicated latch instead of another thread's internal state.
> TestBlockManager#testBlockReportQueueing fails intermittently
> -------------------------------------------------------------
>
> Key: HDFS-10641
> URL: https://issues.apache.org/jira/browse/HDFS-10641
> Project: Hadoop HDFS
> Issue Type: Sub-task
> Components: namenode, test
> Reporter: Mingliang Liu
> Assignee: Mingliang Liu
> Attachments: HDFS-10641.000.patch, HDFS-10641.001.patch
>
>
> See example failing [stack
> trace|https://builds.apache.org/job/PreCommit-HADOOP-Build/9996/testReport/org.apache.hadoop.hdfs.server.blockmanagement/TestBlockManager/testBlockReportQueueing/].
> h6. Stacktrace
> {code}
> java.lang.AssertionError: null
> at org.junit.Assert.fail(Assert.java:86)
> at org.junit.Assert.assertTrue(Assert.java:41)
> at org.junit.Assert.assertTrue(Assert.java:52)
> at
> org.apache.hadoop.hdfs.server.blockmanagement.TestBlockManager.testBlockReportQueueing(TestBlockManager.java:1074)
> {code}
> Recent build test failure
> [1|https://builds.apache.org/job/PreCommit-HADOOP-Build/10100/testReport/]
> and [2|https://builds.apache.org/job/PreCommit-HDFS-Build/16229/testReport/]
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
