datafusion-contrib / datafusion-table-providers

DataFusion TableProviders for reading data from other systems
Apache License 2.0
59 stars 16 forks source link

Use DataFusion unparser instead of sql_provider_datafusion/expr.rs #18

Closed phillipleblanc closed 1 month ago

phillipleblanc commented 3 months ago

There is basic support for converting DataFusion expr to SQL as part of https://github.com/datafusion-contrib/datafusion-table-providers/blob/main/src/sql/sql_provider_datafusion/expr.rs

We should just be using the DataFusion unparser directly and remove this crate's version.

hozan23 commented 1 month ago

@phillipleblanc I think we can close this issue ref: #99, #102