You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@orc.apache.org by "David Mollitor (Jira)" <ji...@apache.org> on 2021/07/07 16:59:00 UTC

[jira] [Created] (ORC-836) StringGroupFromDoubleTreeReader Use Long.toString

David Mollitor created ORC-836:
----------------------------------

             Summary: StringGroupFromDoubleTreeReader Use Long.toString
                 Key: ORC-836
                 URL: https://issues.apache.org/jira/browse/ORC-836
             Project: ORC
          Issue Type: Improvement
            Reporter: David Mollitor
            Assignee: David Mollitor


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.3.4#803005)