Github user omalley commented on a diff in the pull request:

    https://github.com/apache/orc/pull/245#discussion_r181525462
  
    --- Diff: site/_docs/encodings.md ---
    @@ -122,6 +132,12 @@ DIRECT        | PRESENT         | Yes      | Boolean 
RLE
     DIRECT_V2     | PRESENT         | Yes      | Boolean RLE
                   | DATA            | No       | Unbounded base 128 varints
                   | SECONDARY       | No       | Unsigned Integer RLE v2
    +DECIMAL_V1    | PRESENT         | Yes      | Boolean RLE
    +              | DATA            | No       | Signed Integer RLE v1
    +              | SECONDARY       | Yes      | Signed Integer RLE v1
    --- End diff --
    
    We probably need a RLE128 that can just encode a int128 directly. Then we 
can just use the DATA stream directly.


---

Reply via email to