[ https://issues.apache.org/jira/browse/HDFS-10626?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Yiqun Lin updated HDFS-10626: ----------------------------- Description: VolumeScanner throws incorrect IOException in {{datanode.reportBadBlocks}}. The related codes: {code} public void handle(ExtendedBlock block, IOException e) { FsVolumeSpi volume = scanner.volume; ... try { scanner.datanode.reportBadBlocks(block, volume); } catch (IOException ie) { // This is bad, but not bad enough to shut down the scanner. LOG.warn("Cannot report bad " + block.getBlockId(), e); } } {code} The IOException that printed in the log should be {{ie}} rather than {{e}} which was passed in method {{handle(ExtendedBlock block, IOException e)}}. It will be a important info that can help us to know why datanode reporBadBlocks failed. was: VolumeScanner throws incorrect IOException in {{datanode.reportBadBlocks}}. The related codes: {code} public void handle(ExtendedBlock block, IOException e) { FsVolumeSpi volume = scanner.volume; ... try { scanner.datanode.reportBadBlocks(block, volume); } catch (IOException ie) { // This is bad, but not bad enough to shut down the scanner. LOG.warn("Cannot report bad " + block.getBlockId(), e); } } {code} The IOException that printed in the log should be {{ie}} rather than {{e}} which was passed in method {{handle(ExtendedBlock block, IOException e)}}. > VolumeScanner prints incorrect IOException in reportBadBlocks operation > ----------------------------------------------------------------------- > > Key: HDFS-10626 > URL: https://issues.apache.org/jira/browse/HDFS-10626 > Project: Hadoop HDFS > Issue Type: Bug > Reporter: Yiqun Lin > Assignee: Yiqun Lin > Priority: Minor > Attachments: HDFS-10626.001.patch > > > VolumeScanner throws incorrect IOException in {{datanode.reportBadBlocks}}. > The related codes: > {code} > public void handle(ExtendedBlock block, IOException e) { > FsVolumeSpi volume = scanner.volume; > ... > try { > scanner.datanode.reportBadBlocks(block, volume); > } catch (IOException ie) { > // This is bad, but not bad enough to shut down the scanner. > LOG.warn("Cannot report bad " + block.getBlockId(), e); > } > } > {code} > The IOException that printed in the log should be {{ie}} rather than {{e}} > which was passed in method {{handle(ExtendedBlock block, IOException e)}}. > It will be a important info that can help us to know why datanode > reporBadBlocks failed. -- This message was sent by Atlassian JIRA (v6.3.4#6332) --------------------------------------------------------------------- To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org