Github user rip-nsk commented on a diff in the pull request:
https://github.com/apache/orc/pull/212#discussion_r162753893
--- Diff: c++/src/RLEv1.cc ---
@@ -148,7 +148,7 @@ void RleEncoderV1::write(int64_t value) {
numLiterals += 1;
} else {
numLiterals -= static_cast<int>(MINIMUM_REPEAT - 1);
- long base = literals[numLiterals];
+ int64_t base = literals[numLiterals];
--- End diff --
'auto' is more suitable, but it seems its usage is avoided here---
