You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@reef.apache.org by "Yunseong Lee (JIRA)" <ji...@apache.org> on 2016/07/18 11:52:20 UTC

[jira] [Commented] (REEF-967) Handle null in RequiredImpl

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

Yunseong Lee commented on REEF-967:
-----------------------------------

[~jieunparklee] has volunteered to work on this issue! Thanks!

> Handle null in RequiredImpl
> ---------------------------
>
>                 Key: REEF-967
>                 URL: https://issues.apache.org/jira/browse/REEF-967
>             Project: REEF
>          Issue Type: Sub-task
>          Components: Tang
>            Reporter: Sergey Dudoladov
>            Assignee: Ji Eun Lee
>            Priority: Minor
>
>  Setting {{RequiredImpl}} to null should throw an informative exception instead of NPE. 
> To reproduce, in the {{TestConfigurationModule}} set the {{RequiredImpl<TestConfigurationModule.Foo> THE_FOO}} to null:
> {code}
>  String s = null;
>  MyConfigurationModule.CONF
>          .set(MyConfigurationModule.THE_FOO, s)
>          .build();
> {code}
> This code raises {{java.lang.NullPointerException}}  in {{org.apache.reef.tang.util.ReflectionUtilities.classForName(ReflectionUtilities.java:170)}}



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