[ https://issues.apache.org/jira/browse/HADOOP-13665?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15768448#comment-15768448 ]
Wei-Chiu Chuang commented on HADOOP-13665: ------------------------------------------ [~kaisasak] thanks for the patch! The patch looks good overall. I would like to ask you to add a few log message for better supportability. * Please log a debug message for the codec that is eventually loaded. * Inside {{catch (RuntimeException e) }} block, please add either a WARN log that prints the cause of the exception. These messages can greatly help diagnosing performance issues. > Erasure Coding codec should support fallback coder > -------------------------------------------------- > > Key: HADOOP-13665 > URL: https://issues.apache.org/jira/browse/HADOOP-13665 > Project: Hadoop Common > Issue Type: Sub-task > Components: io > Reporter: Wei-Chiu Chuang > Assignee: Kai Sasaki > Labels: hdfs-ec-3.0-must-do > Attachments: HADOOP-13665.01.patch, HADOOP-13665.02.patch > > > The current EC codec supports a single coder only (by default pure Java > implementation). If the native coder is specified but is unavailable, it > should fallback to pure Java implementation. > One possible solution is to follow the convention of existing Hadoop native > codec, such as transport encryption (see {{CryptoCodec.java}}). It supports > fallback by specifying two or multiple coders as the value of property, and > loads coders in order. -- This message was sent by Atlassian JIRA (v6.3.4#6332) --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org