Remove custom sqlparser expr visitor in favour of the native one

splitgraph / seafowl

Analytical database for data-driven Web applications 🪶

https://seafowl.io

Apache License 2.0

388 stars 9 forks source link

Remove custom sqlparser expr visitor in favour of the native one #466

Closed gruuya closed 7 months ago

gruuya commented 7 months ago

Delete some boilerplate code (which typically requires intervention when upgrading DataFusion) since we can now pick up changes from https://github.com/sqlparser-rs/sqlparser-rs/pull/951. This in turn allows us to use the built-in TableFactor visitor method to perform time-travel parsing.