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

          Activity

          Hide
          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
          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
          leftylev Lefty Leverenz added a comment -

          Any doc needed?

          Show
          leftylev Lefty Leverenz added a comment - Any doc needed?
          Hide
          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