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)