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

William Hyun closed ORC-836.
----------------------------

> StringGroupFromDoubleTreeReader Use Double toString
> ---------------------------------------------------
>
>                 Key: ORC-836
>                 URL: https://issues.apache.org/jira/browse/ORC-836
>             Project: ORC
>          Issue Type: Improvement
>    Affects Versions: 1.8.0
>            Reporter: David Mollitor
>            Assignee: David Mollitor
>            Priority: Minor
>             Fix For: 1.8.0
>
>
> https://github.com/apache/orc/blob/fb0ed31f15d9b77b76bc5dec8721f5d9ee129879/java/core/src/java/org/apache/orc/impl/ConvertTreeReaderFactory.java#L1104-L1106
> {code:java}
>       if (!Double.isNaN(doubleValue)) {
>         String string = String.valueOf(doubleValue);
>         byte[] bytes = string.getBytes(StandardCharsets.UTF_8);
>         assignStringGroupVectorEntry(bytesColVector, elementNum, readerType, 
> bytes);
>       }
> {code}
> {{String.valueOf}} simply calls {{Double.toString()}}, so just skip the jumps 
> and call {{Double.toString()}} directly.
>  Also, does not need to be {{UTF_8}}.  Will only ever be ASCI values.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to