It looks like the review is converging, so hopefully we can get this in soon and people can play around with it in their development environments some more.
Also, I wanted to call out all of the bugs that Quanlong has found and been fixing in the ORC C++ library - he's found a whole bunch of pre-existing issues as a result of the testing done on this patch: https://issues.apache.org/jira/issues/?jql=project%20%3D%20ORC%20AND%20reporter%20in%20(stiga-huang) On Mon, Feb 12, 2018 at 1:52 PM, Tim Armstrong <tarmstr...@cloudera.com> wrote: > Maybe it would make sense to create an Epic in JIRA for ORC scanner > enhancements, following on from the initial implementation. I don't really > feel strongly as long as the related JIRAs are linked together somehow. > > On Mon, Feb 12, 2018 at 1:42 PM, Quanlong Huang <huang_quanl...@126.com> > wrote: > >> Dimitris, as the first step, this patch only supports reading primitive >> types from ORC files. I just created two follow-up JIRAs for reading >> complex types (IMPALA-6503) and writing to ORC tables (IMPALA-6504). Will >> work on them later. >> >> Tim, I also created some follow-on JIRAs as you suggest in the comments. >> Should I create a parent JIRA for these ORC tickets, or use IMPALA-5717 as >> the parent JIRA? >> >> >> At 2018-02-13 02:26:10, "Dimitris Tsirogiannis" < >> dtsirogian...@cloudera.com> wrote: >> >Does the patch also implement an ORC writer? >> > >> >Dimitris >> > >> >On Mon, Feb 12, 2018 at 8:48 AM, Jim Apple <jbap...@cloudera.com> wrote: >> > >> >> I agree with the previous comments on this thread. Thank you for >> >> contributing, Quanlong! >> >> >> > >