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

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2020/12/27 16:17:53 UTC

[GitHub] [arrow] mathyingzhou commented on pull request #8648: ARROW-7906: [C++] Add ORC write support

mathyingzhou commented on pull request #8648:
URL: https://github.com/apache/arrow/pull/8648#issuecomment-751486530


   I have revamped the tests completely and refactored the code to eliminate dependency issues and get all checks to pass. Right now I'm integrating my old nested type tests into adapter_test.cpp. Then I will make this PR ready for review again.
   
   Note that support for dense union and sparse union has been delayed till a further PR since there is no read union in the ORC reader which makes testing hard. In that PR I will probably add the following features:
   
   1. Read and write support for union types.
   2. Replicate orc::WriterOptions in adapters::orc::ORCWriterOptions.
   3. Replicate orc::ReaderOptions in adapters::orc::ORCReaderOptions and add them to the the ORC reader.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org