PHOENIX-6450 Checkstyle creating warnings for line length > 80 but < 100
Change-Id: I24ae68bb3c237e1327b2c93774727f328b11aa50
diff --git a/phoenix-assembly/pom.xml b/phoenix-assembly/pom.xml
index 6b4deff..ac1897a 100644
--- a/phoenix-assembly/pom.xml
+++ b/phoenix-assembly/pom.xml
@@ -30,7 +30,6 @@
<packaging>pom</packaging>
<properties>
- <top.dir>${project.basedir}/..</top.dir>
<maven.test.skip>true</maven.test.skip>
<source.skip>true</source.skip>
</properties>
diff --git a/phoenix-client-parent/pom.xml b/phoenix-client-parent/pom.xml
index 5d5a3e1..9a27aea 100644
--- a/phoenix-client-parent/pom.xml
+++ b/phoenix-client-parent/pom.xml
@@ -34,7 +34,6 @@
<!-- Don't make a source-jar -->
<source.skip>true</source.skip>
<license.bundles.dependencies>true</license.bundles.dependencies>
- <top.dir>${project.basedir}/..</top.dir>
<shaded.package>org.apache.phoenix.shaded</shaded.package>
</properties>
diff --git a/phoenix-core/pom.xml b/phoenix-core/pom.xml
index 7b17091..1498012 100644
--- a/phoenix-core/pom.xml
+++ b/phoenix-core/pom.xml
@@ -27,10 +27,6 @@
<name>Phoenix Core</name>
<description>Core Phoenix codebase</description>
- <properties>
- <top.dir>${project.basedir}/..</top.dir>
- </properties>
-
<build>
<plugins>
<!-- Add the ant-generated sources to the source path -->
diff --git a/phoenix-pherf/pom.xml b/phoenix-pherf/pom.xml
index f5ba83b..4ba9260 100644
--- a/phoenix-pherf/pom.xml
+++ b/phoenix-pherf/pom.xml
@@ -30,7 +30,6 @@
<name>Phoenix - Pherf</name>
<properties>
- <top.dir>${project.basedir}/..</top.dir>
<shaded.package>org.apache.phoenix.shaded</shaded.package>
<!-- Versions for pherf-only dependencies -->
diff --git a/phoenix-server/pom.xml b/phoenix-server/pom.xml
index 483bd19..ea30e91 100644
--- a/phoenix-server/pom.xml
+++ b/phoenix-server/pom.xml
@@ -36,7 +36,6 @@
<!-- Don't make a source-jar -->
<source.skip>true</source.skip>
<license.bundles.dependencies>true</license.bundles.dependencies>
- <top.dir>${project.basedir}/..</top.dir>
<shaded.package>org.apache.phoenix.shaded</shaded.package>
</properties>
diff --git a/phoenix-tracing-webapp/pom.xml b/phoenix-tracing-webapp/pom.xml
index 3b44308..e93b530 100755
--- a/phoenix-tracing-webapp/pom.xml
+++ b/phoenix-tracing-webapp/pom.xml
@@ -30,8 +30,6 @@
<description>Tracing web application will visualize the phoenix traces</description>
<properties>
- <top.dir>${project.basedir}/..</top.dir>
-
<!-- tracing webapp only versions -->
<servlet.api.version>3.1.0</servlet.api.version>
<jetty.version>9.3.27.v20190418</jetty.version>
diff --git a/pom.xml b/pom.xml
index da48ffc..aceb538 100644
--- a/pom.xml
+++ b/pom.xml
@@ -99,7 +99,7 @@
<antlr-input.dir>src/main/antlr3</antlr-input.dir>
<antlr-output.dir>target/generated-sources/antlr3</antlr-output.dir>
<test.output.tofile>true</test.output.tofile>
- <top.dir>${project.basedir}</top.dir>
+ <top.dir>${project.basedir}/..</top.dir>
<!-- Dependency versions -->
<jackson.version>2.10.3</jackson.version>
@@ -495,18 +495,20 @@
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-checkstyle-plugin</artifactId>
<version>3.1.0</version>
+ <configuration>
+ <configLocation>${top.dir}/src/main/config/checkstyle/checker.xml</configLocation>
+ <suppressionsLocation>${top.dir}/src/main/config/checkstyle/suppressions.xml</suppressionsLocation>
+ <consoleOutput>true</consoleOutput>
+ <headerLocation>${top.dir}/src/main/config/checkstyle/header.txt</headerLocation>
+ <failOnViolation><!--true-->false</failOnViolation>
+ <includeTestSourceDirectory><!--true-->false</includeTestSourceDirectory>
+ </configuration>
<executions>
<execution>
<id>validate</id>
<phase>validate</phase>
<configuration>
<skip>true</skip>
- <configLocation>${top.dir}/src/main/config/checkstyle/checker.xml</configLocation>
- <suppressionsLocation>${top.dir}/src/main/config/checkstyle/suppressions.xml</suppressionsLocation>
- <consoleOutput>true</consoleOutput>
- <headerLocation>${top.dir}/src/main/config/checkstyle/header.txt</headerLocation>
- <failOnViolation><!--true-->false</failOnViolation>
- <includeTestSourceDirectory><!--true-->false</includeTestSourceDirectory>
</configuration>
<goals>
<goal>check</goal>