Oleks V
Oleks V
@andygrove @advancedxy I fixed the test, implementing extra parsing and manual small tests if the parsing is complicated. I hope now we have better picture.
Thanks @advancedxy I fixed the flaws you mentioned. However I'd like to make refactoring you recommended in followup PR, this PR getting too large for review
@andygrove I fixed all the comments, however you are right, sometimes we support partially the function. means part of syntax or some value range not supported. here comes an idea...
DF still returns formatted messages, `physical_plan.execute(...)?` will produce a formatted error message so we still have to live with that. If we looking for sending the Comet message to the...
My naive approach was to fix the formatter and for some alignment case the prefix won't be added ``` println!("{}", err) => Execution error: err println!("{:>}", err) => err ```...
Filed https://github.com/apache/datafusion/pull/10186
@andygrove we the proposed by @alamb approach we can do something like coded in https://play.rust-lang.org/?version=stable&mode=debug&edition=2021&gist=4d0e0462ab6db7fa63e7791e941eea36 Notice `Exec err:` is gone. Btw this is another question on why we do chain...
Depends on DF 38.0.0
depends on https://github.com/apache/arrow-datafusion/pull/9686
Closing it as this PR was moved to #599