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

Doug Cutting commented on HADOOP-3208:
--------------------------------------

The patch looks good to me.  Can you please add a test case?

Should this be a blocker?  It could break Configurable Writable implementations 
that depend on a non-null configuration.

> WritableDeserializer does not pass the Configuration to deserialized Writables
> ------------------------------------------------------------------------------
>
>                 Key: HADOOP-3208
>                 URL: https://issues.apache.org/jira/browse/HADOOP-3208
>             Project: Hadoop Core
>          Issue Type: Bug
>    Affects Versions: 0.17.0
>            Reporter: Enis Soztutar
>            Assignee: Enis Soztutar
>             Fix For: 0.17.0
>
>         Attachments: writableDeserializer_v1.patch
>
>
> {{WritableDeserializer}} does not hold a reference to the Configuration 
> object, so it does not pass it to deserialized writables, but uses 
> ReflectionUtils.newInstance(writableClass, null), which results in objects 
> defining Configurable interface to be initialized with null configuration. 
> Serialization framework have been introduced in 0.17, so this bug affects 
> 0.17. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to