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

Harsh J commented on HADOOP-8415:
---------------------------------

There's no reason we can't add it. However, have you a need for always using 
doubles (note that you can still use them via set(str, str), which is what 
happens in the end anyway) in your app code?

I've had a need once before, but in my case falling back to a float cast was 
OK: 
https://issues.apache.org/jira/browse/MAPREDUCE-2170?focusedCommentId=12927465&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-12927465

Thanks for your patch against 1.x, can you please provide a patch against 
trunk, and also add test cases that test this new addition?
                
> getDouble() and setDouble() in org.apache.hadoop.conf.Configuration
> -------------------------------------------------------------------
>
>                 Key: HADOOP-8415
>                 URL: https://issues.apache.org/jira/browse/HADOOP-8415
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: conf
>    Affects Versions: 1.0.2
>            Reporter: Jan van der Lugt
>            Priority: Minor
>         Attachments: HADOOP-8415.patch
>
>   Original Estimate: 0.25h
>  Remaining Estimate: 0.25h
>
> In the org.apache.hadoop.conf.Configuration class, methods exist to set 
> Integers, Longs, Booleans, Floats and Strings, but methods for Doubles are 
> absent. Are they not there for a reason or should they be added? In the 
> latter case, the attached patch contains the missing functions.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to