[
https://issues.apache.org/jira/browse/HDFS-6783?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Yi Liu updated HDFS-6783:
-------------------------
Attachment: HDFS-6783.006.patch
Thanks [~cmccabe], I agree with the logic in the new patch. One small nit:
initial value of {{neededScanCount}} should be 0, right? We don’t need to
trigger a {{needsRescan}} at initial state, I update it in the new version.
> Fix HDFS CacheReplicationMonitor rescan logic
> ---------------------------------------------
>
> Key: HDFS-6783
> URL: https://issues.apache.org/jira/browse/HDFS-6783
> Project: Hadoop HDFS
> Issue Type: Bug
> Components: caching
> Affects Versions: 3.0.0
> Reporter: Yi Liu
> Assignee: Yi Liu
> Attachments: HDFS-6783.001.patch, HDFS-6783.002.patch,
> HDFS-6783.003.patch, HDFS-6783.004.patch, HDFS-6783.005.patch,
> HDFS-6783.006.patch
>
>
> In monitor thread, needsRescan is set to false before real scan starts, so
> for {{waitForRescanIfNeeded}} will return for the first condition:
> {code}
> if (!needsRescan) {
> return;
> }
> {code}
--
This message was sent by Atlassian JIRA
(v6.2#6252)
