Andrew Lamb

Results 1844 comments of 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