[jira] [Commented] (ACCUMULO-3245) Property.createInstanceFromPropertyName doesn't validate nulls

["Christopher Tubbs (JIRA)" <ji...@apache.org>]

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

Christopher Tubbs commented on ACCUMULO-3245:
---------------------------------------------

Not sure how this is possible. Any property on which this is used should have a default value in the DefaultConfiguration if it's value is null. If that property doesn't have a default value, for whatever reason, then we probably don't want to try to instantiate it without checking its value first.

> Property.createInstanceFromPropertyName doesn't validate nulls
> --------------------------------------------------------------
>
>                 Key: ACCUMULO-3245
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-3245
>             Project: Accumulo
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 1.6.2
>            Reporter: John Vines
>
> I am using Property.createInstanceFromPropertyName in code which is not running via the accumulo start classloader so it's not being configured (may not be relevent). I get the following error logged-
> {code}2014-10-21 12:44:23,423 [conf.Property] WARN : Failed to load class 
> java.lang.NullPointerException
> 	at org.apache.accumulo.start.classloader.AccumuloClassLoader$2.loadClass(AccumuloClassLoader.java:261)
> 	at java.lang.ClassLoader.loadClass(ClassLoader.java:316)
> 	at java.lang.ClassLoader.loadClass(ClassLoader.java:268)
> 	at org.apache.accumulo.start.classloader.vfs.AccumuloVFSClassLoader.loadClass(AccumuloVFSClassLoader.java:112)
> 	at org.apache.accumulo.core.conf.Property.createInstance(Property.java:641)
> 	at org.apache.accumulo.core.conf.Property.createInstanceFromPropertyName(Property.java:668){code}
> This is considered public code and should probably have better handling for null configurations



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