Andrew Lamb
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
cc @Blizzara and @waynexia
@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...