Andrew Lamb

Results 1689 comments of Andrew Lamb

We hit this in arrow-rs as well https://github.com/apache/arrow-rs/issues/5565

> [@alamb](https://github.com/alamb), you might have some thoughts on this. We'd like to eventually expose the Parquet encryption options in delta-rs, which looks like it would first require integration in Datafusion....

Related discussions about recursion limits: - [ ] https://github.com/apache/datafusion-sqlparser-rs/issues/305 - [ ] https://github.com/apache/datafusion-sqlparser-rs/issues/984 https://docs.rs/sqlparser/latest/sqlparser/parser/struct.Parser.html#method.with_recursion_limit

It seems like we decided to go the `stacker` route in DataFusion with the rationale descrsibed here: https://github.com/apache/datafusion/pull/13177#issuecomment-2465457730 Perhaps it is worth revisiting the use of stacker in this library...

> I have add `stacker` as optional dependency, so it won't be used in no std environment. > > *It will continue to stack overflow in this case. > >...

I am sorry I haven't had a chance to review this yet. It would be great if @mbutrovich could also take a look. I have this on my list to...

I merged up to resolve a conflict

https://github.com/apache/datafusion/actions/runs/15936215612/job/44956521967?pr=16351 That is a new failure for me ``` Running bin/sqllogictests.rs (target/ci/deps/sqllogictests-19519efa72c14141) External error: 1 errors in file /Users/runner/work/datafusion/datafusion/datafusion/sqllogictest/test_files/union_by_name.slt 1. query is expected to fail, but actually succeed: [SQL] select...

Thanks again @corwinjoy / @adamreeve and everyone else. This is great