caicancai commented on PR #3787: URL: https://github.com/apache/calcite/pull/3787#issuecomment-2098514115
> looks good to me in general. just one question, does this exception have to be thrown at runtime? i noticed that the exception is thrown directly in SqlFunctions. @mihaibudiu Thank you for your review. I write this because spark has this mechanism, but calcite is usually checked in Opera and Checker. Which method do you think should be used? -- 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: commits-unsubscr...@calcite.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org