You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by ab...@apache.org on 2019/07/04 07:07:19 UTC

[hive] branch master updated: HIVE-21941: Use checkstyle ruleset in Pre Upgrade Tool project (Krisztian Kasa via Laszlo Bodor)

This is an automated email from the ASF dual-hosted git repository.

abstractdog pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hive.git


The following commit(s) were added to refs/heads/master by this push:
     new d8714e0  HIVE-21941: Use checkstyle ruleset in Pre Upgrade Tool project (Krisztian Kasa via Laszlo Bodor)
d8714e0 is described below

commit d8714e0906e2d81fa0a0bb453e6da01c3e1cb3d1
Author: Krisztian Kasa <kk...@cloudera.com>
AuthorDate: Thu Jul 4 09:06:59 2019 +0200

    HIVE-21941: Use checkstyle ruleset in Pre Upgrade Tool project (Krisztian Kasa via Laszlo Bodor)
    
    Signed-off-by: Laszlo Bodor <bo...@gmail.com>
---
 upgrade-acid/pom.xml | 15 ++++++++++++++-
 1 file changed, 14 insertions(+), 1 deletion(-)

diff --git a/upgrade-acid/pom.xml b/upgrade-acid/pom.xml
index 39f36db..1ec720f 100644
--- a/upgrade-acid/pom.xml
+++ b/upgrade-acid/pom.xml
@@ -46,12 +46,25 @@
         <!-- Plugin versions -->
         <ant.contrib.version>1.0b3</ant.contrib.version>
         <maven.antrun.plugin.version>1.7</maven.antrun.plugin.version>
-        <checkstyle.conf.dir>${basedir}/checkstyle/</checkstyle.conf.dir>
+        <checkstyle.conf.dir>${basedir}/${hive.path.to.root}/checkstyle</checkstyle.conf.dir>
         <maven.checkstyle.plugin.version>2.17</maven.checkstyle.plugin.version>
         <maven.surefire.version>2.20.1</maven.surefire.version>
     </properties>
     <modules>
         <module>pre-upgrade</module>
     </modules>
+    <build>
+        <plugins>
+            <plugin>
+                <groupId>org.apache.maven.plugins</groupId>
+                <artifactId>maven-checkstyle-plugin</artifactId>
+                <configuration>
+                    <configLocation>${checkstyle.conf.dir}/checkstyle.xml</configLocation>
+                    <propertyExpansion>config_loc=${checkstyle.conf.dir}</propertyExpansion>
+                    <includeTestSourceDirectory>true</includeTestSourceDirectory>
+                </configuration>
+            </plugin>
+        </plugins>
+    </build>
 
 </project>
\ No newline at end of file