Ankur Goyal
Ankur Goyal
@alamb @jakeswenson yes indeed! I posted this above too, but it'd be great to understand the differences between this and #790. In general, I'd prefer to collaborate on that PR...
Happy to contribute this @alamb @AugustoFKL
@alamb I'm happy to add a PR for this if that'd be helpful
Here you go for DataFusion: https://github.com/apache/arrow-datafusion/compare/main...qscl:arrow-datafusion:sqlparser. I used [this branch](https://github.com/qscl/sqlparser-rs/tree/datafusion) which is approximately the same as the PR (more recently rebased + fixed support for `Visitor`). I verified that on...
Would it be worthwhile to chat live? I know you're tight on time, but may be useful to brainstorm a few ideas. Some things that come to mind for me...
Of note -- this does not repro in the python library or duckdb binary.
Interesting, on my machine, both in the CLI and Python lib for 0.8 and 0.9, it completes instantly.
It also does not seem to repro for me on duckdb-wasm using duckdb 0.8
I also benchmarked it using `time -l` on duckdb 0.9.1 vs duckdb 0.8.1 on the CLI: 0.9.1: ``` 19333120 maximum resident set size 0 average shared memory size 0 average...