[jira] [Created] (HDFS-10089) CLONE - Erasure Coding: Few improvements for the erasure coding worker

["dragon (JIRA)" <ji...@apache.org>]

dragon created HDFS-10089:
-----------------------------

             Summary: CLONE - Erasure Coding: Few improvements for the erasure coding worker
                 Key: HDFS-10089
                 URL: https://issues.apache.org/jira/browse/HDFS-10089
             Project: Hadoop HDFS
          Issue Type: Sub-task
            Reporter: dragon
            Assignee: Rakesh R
            Priority: Minor
             Fix For: HDFS-7285


# Following log is confusing, make it tidy. Its missing {{break;}} statement and causing this unwanted logs.
{code}
2015-05-10 15:06:45,878 INFO  datanode.DataNode (BPOfferService.java:processCommandFromActive(728)) - DatanodeCommand action: DNA_ERASURE_CODING_RECOVERY
2015-05-10 15:06:45,879 WARN  datanode.DataNode (BPOfferService.java:processCommandFromActive(732)) - Unknown DatanodeCommand action: 11
{code}
# Add exception trace to the log, would improve debuggability
{code}
} catch (Throwable e) {
   LOG.warn("Failed to recover striped block: " + blockGroup);
}
{code}
# Make member variables present in ErasureCodingWorker, ReconstructAndTransferBlock, StripedReader {{private}} {{final}}
# Correct spelling of the variable {{STRIPED_READ_TRHEAD_POOL}} to {{STRIPED_READ_THREAD_POOL}}
# Good to add debug logs to print the striped read pool size
{code}
LOG.debug("Using striped reads; pool threads=" + num);
{code}
# Add meaningful message to the precondition check:
{code}
Preconditions.checkArgument(liveIndices.length == sources.length);
{code}
# Remove unused import
{code}
import org.apache.hadoop.hdfs.server.common.HdfsServerConstants;
{code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)