rubenada commented on PR #3865: URL: https://github.com/apache/calcite/pull/3865#issuecomment-2238721010
> Should we add tests to SqlOperatorTest? @caicancai I think there is no need to touch `SqlOperatorTest`. The way that class works, the already existing tests `testLenFunc` and `testLengthFunc` will automatically consider the libraries in the `@LibraryOperator` annotation of `LEN` and `LENGTH` (including the newly added ones). So new checks are added transparently for the new libraries. -- 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]
