blaginin commented on PR #1522: URL: https://github.com/apache/datafusion-sqlparser-rs/pull/1522#issuecomment-2477412434
I was wondering if we should remove RecursionCounter with this PR. In my opinion, we shouldn't, because of: - Backwards compatibility - The ability to limit max recursion is useful, regardless of our ability to handle large queries without crashing. -- 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]
