You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@orc.apache.org by omalley <gi...@git.apache.org> on 2017/09/05 16:34:47 UTC

[GitHub] orc pull request #166: ORC-91: Use hdfs v-blocks instead of zero-padding str...

GitHub user omalley opened a pull request:

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

    ORC-91: Use hdfs v-blocks instead of zero-padding stripes

    Ok, this is my rebasing of Gopal's patch to use the zero padding after my patch for ORC-234.

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

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

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

    https://github.com/apache/orc/pull/166.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 #166
    
----
commit fb3f096e6724811bb9604c016b008519d34288b8
Author: Gopal V <go...@apache.org>
Date:   2017-07-25T04:15:57Z

    ORC-91: Use hdfs v-blocks instead of zero-padding stripes

----


---

[GitHub] orc pull request #166: ORC-91: Use hdfs v-blocks instead of zero-padding str...

Posted by asfgit <gi...@git.apache.org>.

Github user asfgit closed the pull request at:

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


---

[GitHub] orc issue #166: ORC-91: Use hdfs v-blocks instead of zero-padding stripes

Posted by omalley <gi...@git.apache.org>.

Github user omalley commented on the issue:

    https://github.com/apache/orc/pull/166
  
    Changes:
    * Renamed HadoopShims_2_2 to HadoopShimsPre2_3 and moved to shims module.
    * Removed TextReaderShim since we aren't using it any more.
    * Pulled the padding method into the HadoopShim as padStreamToBlock.
    * Added HadoopShimsPre2_7.
    * Refactored a bit so that the shim classes share most of the code, while making sure that older shims never reference new shims.
    * Fixed some broken javadoc.


---