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

    https://github.com/apache/orc/pull/212#discussion_r163463896
  
    --- Diff: c++/include/orc/Common.hh ---
    @@ -69,7 +69,7 @@ namespace orc {
         UNKNOWN_WRITER = INT32_MAX
       };
     
    -  enum CompressionKind {
    +  enum CompressionKind : std::int64_t {
    --- End diff --
    
    We can potentially get rid of this enum and use proto::CompressionKind 
instead. It can be done in a separate change.


---

Reply via email to