Andrew Lamb

Results 1625 comments of Andrew Lamb

@LeslieKid added time/interval/ decimal/utf8view in https://github.com/apache/datafusion/pull/13226 Additional types that would be good to cover are: 1. `Float32/Float64` 2. `Date` and `Timestamp` Any chance you are interested in doing that too...

> Sorry, I'm currently unable to take that on at the moment. I think filing a ticket is a good way to forward. Thanks @alamb Thank you -- filed https://github.com/apache/datafusion/issues/13279

Now that we have most of the datatypes filled in, perhaps we can start adding coverage for the other aggregation functions. Like `bit_and` etc 🤔

I am closing this as I think we have pretty good fuzz testing now and a framework to extend coverage as needed Thanks everyone

@XiangpengHao -- can you possibly comment / take this issue (I can't assign it to you without you having first commented on it 😆 )

We are done with the draft. We expect it to be published in the next few weeks (it turns out to be a two part series)

In case anyone wants an overview of adding StringView to DataFusion, here is a presentation and slides Video: https://www.youtube.com/watch?v=RVLshX6fbds Slides: https://drive.google.com/file/d/1Qqd8V6cfS9rSQ_-JrinasQJwI79qlUEV/view?usp=drive_link Blog is coming soon

Part 1: https://www.influxdata.com/blog/faster-queries-with-stringview-part-one-influxdb/

Part 2: https://www.influxdata.com/blog/faster-queries-with-stringview-part-two-influxdb/ cc @XiangpengHao I would like to also make a copy of this content on the DataFusion blog as it is entirely about DataFusion and might be easier...