[ https://issues.apache.org/jira/browse/HDFS-12914?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16859250#comment-16859250 ]
star edited comment on HDFS-12914 at 6/8/19 4:09 PM: ----------------------------------------------------- Few comments about your unit tests. # Following codes bypass lease expiration checking logic by removing valid lease id. Better to keep it as it is in running time. {code:java} // Remove full block report lease about dn spyBlockManager.getBlockReportLeaseManager() .removeLease(datanodeDescriptor);{code} 2. Do we really need to response with a RegisterCommand.REGISTER command to client? It's a somewhat heavy command. Should we just let the client know its failure on block report and just try few more times. was (Author: starphin): Few comments about your unit tests. Following codes bypass lease expiration checking logic by removing valid lease id. Better to keep it as it is in running time. {code:java} // Remove full block report lease about dn spyBlockManager.getBlockReportLeaseManager() .removeLease(datanodeDescriptor); {code} > Block report leases cause missing blocks until next report > ---------------------------------------------------------- > > Key: HDFS-12914 > URL: https://issues.apache.org/jira/browse/HDFS-12914 > Project: Hadoop HDFS > Issue Type: Bug > Components: namenode > Affects Versions: 2.8.0, 2.9.2 > Reporter: Daryn Sharp > Assignee: Santosh Marella > Priority: Critical > Attachments: HDFS-12914-branch-2.001.patch, > HDFS-12914-trunk.00.patch, HDFS-12914-trunk.01.patch, HDFS-12914.005.patch, > HDFS-12914.006.patch > > > {{BlockReportLeaseManager#checkLease}} will reject FBRs from DNs for > conditions such as "unknown datanode", "not in pending set", "lease has > expired", wrong lease id, etc. Lease rejection does not throw an exception. > It returns false which bubbles up to {{NameNodeRpcServer#blockReport}} and > interpreted as {{noStaleStorages}}. > A re-registering node whose FBR is rejected from an invalid lease becomes > active with _no blocks_. A replication storm ensues possibly causing DNs to > temporarily go dead (HDFS-12645), leading to more FBR lease rejections on > re-registration. The cluster will have many "missing blocks" until the DNs > next FBR is sent and/or forced. -- This message was sent by Atlassian JIRA (v7.6.3#76005) --------------------------------------------------------------------- To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org