Github user majetideepak commented on the issue:
https://github.com/apache/orc/pull/273
The PR looks overall good to me apart from a minor change requested. This
is an important patch to align the C++ and Java implementations. Thanks again
for working on this!
---
Github user majetideepak commented on the issue:
https://github.com/apache/orc/pull/275
Since there is some investigation needed here, I am going to merge this
patch. We can enable `NO_SASL` build in a later patch. Right now, this is
causing a build failure by default.
---
Github user majetideepak commented on a diff in the pull request:
https://github.com/apache/orc/pull/273#discussion_r192593861
--- Diff: c++/src/RLEv2.hh ---
@@ -25,13 +25,89 @@
#include
+#define MIN_REPEAT 3
+#define HIST_LEN 32
namespace orc {
Github user asfgit closed the pull request at:
https://github.com/apache/orc/pull/275
---
GitHub user majetideepak opened a pull request:
https://github.com/apache/orc/pull/277
ORC-372: Enable valgrind for C++ travis-ci tests
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/majetideepak/orc ORC-372
Alternatively you
GitHub user omalley opened a pull request:
https://github.com/apache/orc/pull/278
ORC-251: Extend InStream and OutStream to support encryption.
This patch:
* Adds a method to Codec to get the CompressionKind.
* Creates StreamOptions for both InStream and OutStream to gather
Github user majetideepak commented on the issue:
https://github.com/apache/orc/pull/277
There are indeed valgrind failures. I will push a followup patch to fix
these.
---
Wes McKinney created ORC-374:
Summary: Possible to reduce size of release tarballs?
Key: ORC-374
URL: https://issues.apache.org/jira/browse/ORC-374
Project: ORC
Issue Type: Improvement
Github user yuruiz commented on a diff in the pull request:
https://github.com/apache/orc/pull/273#discussion_r192614780
--- Diff: c++/src/RLEv2.hh ---
@@ -25,13 +25,89 @@
#include
+#define MIN_REPEAT 3
+#define HIST_LEN 32
namespace orc {