Mailing-List: contact common-issues-help@hadoop.apache.org; run by ezmlm
Precedence: bulk
Reply-To: common-issues@hadoop.apache.org
Date: Fri, 31 Aug 2012 10:41:07 +1100 (NCT)
From: "Bo Wang (JIRA)" <jira@apache.org>
To: common-issues@hadoop.apache.org
Message-ID: <1997718253.19182.1346370067629.JavaMail.jiratomcat@arcas>
In-Reply-To: <1434097047.19173.1346369948334.JavaMail.jiratomcat@arcas>
Subject: [jira] [Updated] (HADOOP-8750) Configuration.getClasses never
 returns default value
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


     [ https://issues.apache.org/jira/browse/HADOOP-8750?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Bo Wang updated HADOOP-8750:
----------------------------

    Description: 
getClasses method in Configuration never returns defaultValue even if the classnames is not set in Configuration.

public Class<?>[] getClasses(String name, Class<?> ... defaultValue) {
  String[] classnames = getTrimmedStrings(name);
  if (classnames == null)
    return defaultValue;
    // Load and return classes using classnames... (omitted)
}

This is because the condition "if (classnames == null)" is always True. If name is not set in Configuration, classnames will be an empty array (but not null).

To fix it, just change the condition to "if (classnames == null || classnames.length == 0)"


  was:
getClasses method in Configuration never returns defaultValue even if the classnames is not set in Configuration.

public Class<?>[] getClasses(String name, Class<?> ... defaultValue) {
    String[] classnames = getTrimmedStrings(name);
    if (classnames == null)
      return defaultValue;
    // Load and return classes using classnames... (omitted)
  }

This is because the condition "if (classnames == null)" is always True. If name is not set in Configuration, classnames will be an empty array (but not null).

To fix it, just change the condition to "if (classnames == null || classnames.length == 0)"


> Configuration.getClasses never returns default value
> ----------------------------------------------------
>
>                 Key: HADOOP-8750
>                 URL: https://issues.apache.org/jira/browse/HADOOP-8750
>             Project: Hadoop Common
>          Issue Type: Bug
>            Reporter: Bo Wang
>            Assignee: Bo Wang
>
> getClasses method in Configuration never returns defaultValue even if the classnames is not set in Configuration.
> public Class<?>[] getClasses(String name, Class<?> ... defaultValue) {
>   String[] classnames = getTrimmedStrings(name);
>   if (classnames == null)
>     return defaultValue;
>     // Load and return classes using classnames... (omitted)
> }
> This is because the condition "if (classnames == null)" is always True. If name is not set in Configuration, classnames will be an empty array (but not null).
> To fix it, just change the condition to "if (classnames == null || classnames.length == 0)"

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira