|
Just a small hit,
lets change public String[] getStrings(String name, String... defaultValue) { } so that both conf.getStrings(key, new String[] {"foo.bar", "foo.baz"}); and conf.getStrings(key, "foo.bar", "foo.baz"); works. Moreover since we are at it, introducing a setStrings() might be a good idea (the patch for HADOOP-449 needs it). -1 overall. Here are the results of testing the latest attachment
http://issues.apache.org/jira/secure/attachment/12378768/patch-3093.txt against trunk revision 619744. @author +1. The patch does not contain any @author tags. tests included -1. The patch doesn't appear to include any new or modified tests. javadoc +1. The javadoc tool did not generate any warning messages. javac +1. The applied patch does not generate any new javac compiler warnings. release audit +1. The applied patch does not generate any new release audit warnings. findbugs +1. The patch does not introduce any new Findbugs warnings. core tests +1. The patch passed core unit tests. contrib tests +1. The patch passed contrib unit tests. Test results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/2087/testReport/ This message is automatically generated. Canceling the patch to address Enis's suggestion.
Here is a patch adding
1. String[] Configuration.getStrings(String name, String... defaultValue) 2. void Configuration.setStrings(String name, String... values) -1 overall. Here are the results of testing the latest attachment
http://issues.apache.org/jira/secure/attachment/12378920/patch-3093.txt against trunk revision 619744. @author +1. The patch does not contain any @author tags. tests included -1. The patch doesn't appear to include any new or modified tests. patch -1. The patch command could not apply the patch. Console output: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/2099/console This message is automatically generated. trying hudson again.
-1 overall. Here are the results of testing the latest attachment
http://issues.apache.org/jira/secure/attachment/12378920/patch-3093.txt against trunk revision 643282. @author +1. The patch does not contain any @author tags. tests included -1. The patch doesn't appear to include any new or modified tests. javadoc +1. The javadoc tool did not generate any warning messages. javac +1. The applied patch does not generate any new javac compiler warnings. release audit +1. The applied patch does not generate any new release audit warnings. findbugs +1. The patch does not introduce any new Findbugs warnings. core tests +1. The patch passed core unit tests. contrib tests +1. The patch passed contrib unit tests. Test results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/2111/testReport/ This message is automatically generated. I just committed this. Thanks, Amareshwari!
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
I added a new api getStrings(String name, String[] defaultValue) in Configuration.java, which returns default value if nothing specified for the configuration property.