You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-issues@hadoop.apache.org by "Vinayakumar B (JIRA)" <ji...@apache.org> on 2015/04/06 10:58:13 UTC

[jira] [Commented] (HADOOP-11649) Allow to configure multiple erasure codecs

    [ https://issues.apache.org/jira/browse/HADOOP-11649?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14481053#comment-14481053 ] 

Vinayakumar B commented on HADOOP-11649:
----------------------------------------

One question I have is,

1. java ServiceLoader will load all the implementations of specified type in the entire classpath. Right? Will it miss any impl to load?
2. The codes specified via configurations also should be in the classpath to get successfully added. IMO these also will be already loaded by java ServiceLoader. right?
If all the codecs are loaded by ServiceLoader, then there is no need of configuration to add more codecs.

If the above is true, then
configuration to specify codes, can be used as a enabler to use only codecs specified in configuration, not all available in classpath( which are loaded  by ServiceLoader).


Coming to patch,

1. conf name specified in {{public static final String IO_ERASURE_CODECS_KEY = "io.erasure.codecs";}} and {{core-default.xml}} are different.
2. ErasureCodec.java, when there is a explicit dependency mentioned on this Jira, then this class need not be included in the patch.
3. ErasureCodecLoader#load(..) should be changed based on the ans of above question
4. {{io.erasurecode.codec.rs.rawcoder}} adding this configuration seems unrelated to this Jira

> Allow to configure multiple erasure codecs
> ------------------------------------------
>
>                 Key: HADOOP-11649
>                 URL: https://issues.apache.org/jira/browse/HADOOP-11649
>             Project: Hadoop Common
>          Issue Type: Sub-task
>          Components: io
>            Reporter: Kai Zheng
>            Assignee: Kai Zheng
>         Attachments: HADOOP-11649-v1.patch
>
>
> This is to allow to configure erasure codec and coder in core-site configuration file.



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