xudong963 commented on pull request #1437: URL: https://github.com/apache/arrow-datafusion/pull/1437#issuecomment-991856488
> I wonder if it makes sense to just limit number of or statements instead? First, `Statement` represents the SQL AST node, I don't think it's reasonable to limit the depth of recursion stack by limiting the number of `Statement`. Second, in some cases, it's preferable to unconditionally grow the stack than returning err or panic. -- 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]
