Uploaded image for project: 'ORC'
  1. ORC
  2. ORC-81

Add support for lzo and lz4 to c++ reader

        Details

        • Type: Bug
        • Status: Closed
        • Priority: Major
        • Resolution: Fixed
        • Affects Version/s: None
        • Fix Version/s: 1.2.0
        • Component/s: None
        • Labels:
          None

          Description

          Since ORC-77 is adding support for LZO and LZ4 to the Java reader and writer, we need to add matching support for the C++ reader.

          I propose that we port the aircompressor Java LzoDecompressor to C++ so that we end up with an Apache licensed C++ lzo decompressor.

            Issue Links

            links to

            Web Link GitHub Pull Request #47

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              githubbot ASF GitHub Bot added a comment -

              GitHub user omalley opened a pull request:

              https://github.com/apache/orc/pull/47

              ORC-81 Add support for LZO and LZ4 in the C++ reader.

              • Add lz4 to the c++/libs.
              • Add example files for lzo and lz4 from the java writer.
              • Add tests for the new example files and compression codecs.
              • Refactored the snappy codec so that each of the block compressed codecs shares almost all of the code.

              You can merge this pull request into a Git repository by running:

              $ git pull https://github.com/omalley/orc orc-81

              Alternatively you can review and apply these changes as the patch at:

              https://github.com/apache/orc/pull/47.patch

              To close this pull request, make a commit to your master/trunk branch
              with (at least) the following in the commit message:

              This closes #47

              commit f8ae70aa2041cb621c3405c0d83831e148283b83
              Author: Owen O'Malley <omalley@apache.org>
              Date: 2016-07-05T15:16:03Z

              ORC-77. Implement LZO and LZ4 compression codecs.

              commit 199f0c6c375bcf0193c815e4c6ea6ceced0a3f23
              Author: Owen O'Malley <omalley@apache.org>
              Date: 2016-07-07T23:44:07Z

              ORC-81 Add lzo and lz4 support for the C++ reader.

              Show
              githubbot ASF GitHub Bot added a comment - GitHub user omalley opened a pull request: https://github.com/apache/orc/pull/47 ORC-81 Add support for LZO and LZ4 in the C++ reader. Add lz4 to the c++/libs. Add example files for lzo and lz4 from the java writer. Add tests for the new example files and compression codecs. Refactored the snappy codec so that each of the block compressed codecs shares almost all of the code. You can merge this pull request into a Git repository by running: $ git pull https://github.com/omalley/orc orc-81 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/orc/pull/47.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #47 commit f8ae70aa2041cb621c3405c0d83831e148283b83 Author: Owen O'Malley <omalley@apache.org> Date: 2016-07-05T15:16:03Z ORC-77 . Implement LZO and LZ4 compression codecs. commit 199f0c6c375bcf0193c815e4c6ea6ceced0a3f23 Author: Owen O'Malley <omalley@apache.org> Date: 2016-07-07T23:44:07Z ORC-81 Add lzo and lz4 support for the C++ reader.
              Hide
              Permalink
              githubbot ASF GitHub Bot added a comment -

              Github user asfgit closed the pull request at:

              https://github.com/apache/orc/pull/47

              Show
              githubbot ASF GitHub Bot added a comment - Github user asfgit closed the pull request at: https://github.com/apache/orc/pull/47
              Hide
              Permalink
              leftylev Lefty Leverenz added a comment -

              Any doc needed?

              Show
              leftylev Lefty Leverenz added a comment - Any doc needed?
              Hide
              Permalink
              owen.omalley Owen O'Malley added a comment -

              Released as part of ORC 1.2.0

              Show
              owen.omalley Owen O'Malley added a comment - Released as part of ORC 1.2.0

                People

                • Assignee:
                  owen.omalley Owen O'Malley
                  Reporter:
                  owen.omalley Owen O'Malley
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:

                    Development