Github user jamesclampffer commented on a diff in the pull request:
https://github.com/apache/orc/pull/273#discussion_r191932328
--- Diff: c++/include/orc/Writer.hh ---
@@ -38,6 +38,11 @@ namespace orc {
CompressionStrategy_COMPRESSION
};
+ enum RleVersion {
+ RleVersion_1,
--- End diff --
You may want to explicitly assign values here to ensure the addition of new
encodings or refactoring work can't unintentionally change the underlying
integer values.---
