xuzifu666 commented on code in PR #4689:
URL: https://github.com/apache/calcite/pull/4689#discussion_r2626707150


##########
babel/src/test/java/org/apache/calcite/test/BabelTest.java:
##########
@@ -490,4 +496,45 @@ private void checkSqlResult(String funLibrary, String 
query, String result) {
         .query(query)
         .returns(result);
   }
+
+  /**
+   * Test AGE function with timestamp arguments.
+   */
+  @Test void testAgeFunction() {

Review Comment:
   It's generally also recommended to add related tests to 
```SqlOperatorTest``` when adding new functions. Additionally, JIRA link and 
description should be added to the test methods; you can refer to other 
relevant links for more information.



-- 
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]

Reply via email to