hadoop-common-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Nandor Kollar (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HADOOP-14459) SerializationFactory shouldn't throw a NullPointerException if the serializations list is not defined
Date Mon, 29 May 2017 10:10:04 GMT

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

Nandor Kollar commented on HADOOP-14459:
----------------------------------------

[~templedf] thanks for the comments! Attached patch HADOOP-14459_2.patch. It seems that the
original problem (HADOOP-8588) is no longer present with getTrimmedStrings, looks like only
getStrings caused NPE in this case. In the second diff, a warning is emitted if the value
is not defined, or is set to null. If it is undefined, then SerializationFactory will use
the defaults passed to getTrimmedStrings, but for empty value nothing is added. Daniel, what
do you think?

> SerializationFactory shouldn't throw a NullPointerException if the serializations list
is not defined
> -----------------------------------------------------------------------------------------------------
>
>                 Key: HADOOP-14459
>                 URL: https://issues.apache.org/jira/browse/HADOOP-14459
>             Project: Hadoop Common
>          Issue Type: Bug
>            Reporter: Nandor Kollar
>            Assignee: Nandor Kollar
>            Priority: Minor
>         Attachments: HADOOP-14459_2.patch, HADOOP-14459.patch
>
>
> The SerializationFactory throws an NPE if CommonConfigurationKeys.IO_SERIALIZATIONS_KEY
is not defined in the config.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-issues-help@hadoop.apache.org


Mime
View raw message