dawidwys commented on code in PR #24679:
URL: https://github.com/apache/flink/pull/24679#discussion_r1570850917


##########
flink-table/flink-table-planner/src/test/java/org/apache/flink/table/planner/functions/BuiltInAggregateFunctionTestBase.java:
##########
@@ -344,6 +361,38 @@ public void execute(TableEnvironment tEnv, Table 
sourceTable) {
         protected abstract TableResult getResult(TableEnvironment tEnv, Table 
sourceTable);
     }
 
+    private abstract static class RuntimeFailureItem implements TestItem {

Review Comment:
   I followed the `SuccessItem` convention. `Success/Failure`, but sure I'll 
rename it.



-- 
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: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to