[hotfix] Invert activation of new and legacy code by introducing -Dle…

…gacyCode flag

.travis.yml

@@ -39,52 +39,52 @@ matrix:
  - jdk: "oraclejdk8"
  env:
  - TEST="core"
- - PROFILE="-Dhadoop.version=2.8.3 -Dnew"
+ - PROFILE="-Dhadoop.version=2.8.3"
  - CACHE_NAME=JDK8_H280_CO
  - jdk: "oraclejdk8"
  env:
  - TEST="libraries"
- - PROFILE="-Dhadoop.version=2.8.3 -Dnew"
+ - PROFILE="-Dhadoop.version=2.8.3"
  - CACHE_NAME=JDK8_H280_L
  - jdk: "oraclejdk8"
  env:
  - TEST="connectors"
- - PROFILE="-Dhadoop.version=2.8.3 -Dnew -Pinclude-kinesis"
+ - PROFILE="-Dhadoop.version=2.8.3 -Pinclude-kinesis"
  - CACHE_NAME=JDK8_H280_CN
  - jdk: "oraclejdk8"
  env:
  - TEST="tests"
- - PROFILE="-Dhadoop.version=2.8.3 -Dnew"
+ - PROFILE="-Dhadoop.version=2.8.3"
  - CACHE_NAME=JDK8_H280_T
  - jdk: "oraclejdk8"
  env:
  - TEST="misc"
- - PROFILE="-Dhadoop.version=2.8.3 -Dnew -Dinclude_hadoop_aws"
+ - PROFILE="-Dhadoop.version=2.8.3 -Dinclude_hadoop_aws"
  - CACHE_NAME=JDK8_H280_M
  - jdk: "openjdk8"
  env:
  - TEST="core"
- - PROFILE="-Dhadoop.version=2.4.1"
+ - PROFILE="-Dhadoop.version=2.4.1 -DlegacyCode"
  - CACHE_NAME=JDK8_H241_CO
  - jdk: "openjdk8"
  env:
  - TEST="libraries"
- - PROFILE="-Dhadoop.version=2.4.1"
+ - PROFILE="-Dhadoop.version=2.4.1 -DlegacyCode"
  - CACHE_NAME=JDK8_H241_L
  - jdk: "openjdk8"
  env:
  - TEST="connectors"
- - PROFILE="-Dhadoop.version=2.4.1 -Pinclude-kinesis"
+ - PROFILE="-Dhadoop.version=2.4.1 -DlegacyCode -Pinclude-kinesis"
  - CACHE_NAME=JDK8_H241_CN
  - jdk: "openjdk8"
  env:
  - TEST="tests"
- - PROFILE="-Dhadoop.version=2.4.1"
+ - PROFILE="-Dhadoop.version=2.4.1 -DlegacyCode"
  - CACHE_NAME=JDK8_H241_T
  - jdk: "openjdk8"
  env:
  - TEST="misc"
- - PROFILE="-Dhadoop.version=2.4.1"
+ - PROFILE="-Dhadoop.version=2.4.1 -DlegacyCode"
  - CACHE_NAME=JDK8_H241_M
 
 git:

pom.xml

@@ -127,9 +127,7 @@ under the License.
  <powermock.version>1.6.5</powermock.version>
  <hamcrest.version>1.3</hamcrest.version>
  <japicmp.skip>false</japicmp.skip>
- <!-- run all groups except new by default -->
- <test.excludedGroups>org.apache.flink.testutils.category.New</test.excludedGroups>
- <codebase>legacy</codebase>
+ <codebase>new</codebase>
  <!--
  Keeping the MiniKDC version fixed instead of taking hadoop version dependency
  to support testing Kafka, ZK etc., modules that does not have Hadoop dependency
@@ -593,16 +591,14 @@ under the License.
  <profiles>
 
  <profile>
- <id>new</id>
+ <id>legacyCode</id>
  <activation>
  <property>
- <name>new</name>
+ <name>legacyCode</name>
  </property>
  </activation>
  <properties>
- <!-- clear the excluded groups list -->
- <test.excludedGroups></test.excludedGroups>
- <codebase>new</codebase>
+ <codebase>legacy</codebase>
  </properties>
  </profile>
 
@@ -1118,8 +1114,6 @@ under the License.
  <!-- Do NOT use a version >=2.19.X, as test cases may get stuck before execution. See SUREFIRE-1255 -->
  <version>2.18.1</version>
  <configuration>
- <groups>${test.groups}</groups>
- <excludedGroups>${test.excludedGroups}</excludedGroups>
  <forkCount>${flink.forkCount}</forkCount>
  <reuseForks>${flink.reuseForks}</reuseForks>
  <systemPropertyVariables>