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.---
