alamb commented on PR #1522: URL: https://github.com/apache/datafusion-sqlparser-rs/pull/1522#issuecomment-2495966342
> Ah I see, thanks for the context @peter-toth! > > cc @alamb for overall thoughts on adding this dependency to sqlparser? While adding new dependencies in general ls 🤮 I don't think there is any viable alternative in this case We have tried to avoid doing something like `stacker` for several years with sqlparser and datafusion, but I feel like we have only been able to band aid over the problem, not fix the problem once and for all. I am hopeful that if we adopt this particular crate we won't have to worry about it again 🤞 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
