Andrew Lamb
Andrew Lamb
> > @goldmedal would you be ok if I pushed a change to this PR to temporarily patch arrow-rs to include the fix for [apache/arrow-rs#6539](https://github.com/apache/arrow-rs/pull/6539) ? > > Then we...
I am starting to play around with this PR / write some tests. Will post my updates shortly
I am running some benchmarks on this PR
Here is the performance of this PR. Some queries are slower, some are faster. I believe once we turn on string view everything will be faster. ``` -------------------- Benchmark clickbench_partitioned.json...
I reabased / squashed all the code in this branch so it would be easier to pull in to test in https://github.com/apache/datafusion/pull/12092
> > Here is the performance of this PR. Some queries are slower, some are faster. > > I believe once we turn on string view everything will be faster....
Since this PR requires a change to arrow-rs, I think there is no particular rush to merge it in -- I have a few thoughts about how to make the...
I am starting to get this PR ready
Update here is we are on track to release a version of arrow with the required fixes today and then I will merge this PR up and get it ready...
> Thanks, @alamb. I have one question about the test. Others look good to me. Thanks @goldmedal -- that was an excellent catch. I have fixed the issue