orc-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From wgtmac <...@git.apache.org>
Subject [GitHub] orc pull request #245: ORC-161: Proposal for new decimal encodings and stati...
Date Tue, 17 Apr 2018 04:35:42 GMT
Github user wgtmac commented on a diff in the pull request:

    https://github.com/apache/orc/pull/245#discussion_r181950089
  
    --- Diff: site/_docs/file-tail.md ---
    @@ -249,12 +249,25 @@ For booleans, the statistics include the count of false and true
values.
     }
     ```
     
    -For decimals, the minimum, maximum, and sum are stored.
    +For decimals, the minimum, maximum, and sum are stored. In ORC 2.0,
    +string representation is deprecated and DecimalStatistics uses integers
    +which have better performance.
     
     ```message DecimalStatistics {
      optional string minimum = 1;
      optional string maximum = 2;
      optional string sum = 3;
    +  message Int128 {
    --- End diff --
    
    Good suggestion!


---

Mime
View raw message