Github user fhueske commented on a diff in the pull request:

    https://github.com/apache/flink/pull/3511#discussion_r139933797
  
    --- Diff: 
flink-test-utils-parent/flink-test-utils/src/main/java/org/apache/flink/test/util/MultipleProgramsTestBase.java
 ---
    @@ -146,6 +151,7 @@ public static void teardown() throws Exception {
        public static Collection<Object[]> executionModes() {
                return Arrays.asList(
                                new Object[] { TestExecutionMode.CLUSTER },
    +                           new Object[] { 
TestExecutionMode.CLUSTER_WITH_CODEGENERATION_ENABLED },
    --- End diff --
    
    Does this add another execution mode to each tests class that extends 
`MultipleProgramsTestBase`? This would have a big impact on the overall build 
time, right? 
    
    I think it would be better to manually, include add this execution mode to 
a set of selected tests that include sorting.


---

Reply via email to