Re: Re: Re: ORC scanner - points for discussion

2018-03-28 Thread Tim Armstrong
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

Re: Re: ORC scanner - points for discussion

2018-02-12 Thread Tim Armstrong
Putting it behind a flag sounds good to me too. Hopefully we can get feedback from Hulu and other users of Impala that will try out the experimental version. On Mon, Feb 12, 2018 at 10:26 AM, Dimitris Tsirogiannis < dtsirogian...@cloudera.com> wrote: > Does the patch also implement an ORC

Re: Re: ORC scanner - points for discussion

2018-02-12 Thread Jim Apple
I agree with the previous comments on this thread. Thank you for contributing, Quanlong!