You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hudi.apache.org by fo...@apache.org on 2023/01/31 11:05:44 UTC

[hudi] 06/17: improve checkstyle

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

forwardxu pushed a commit to branch release-0.12.1
in repository https://gitbox.apache.org/repos/asf/hudi.git

commit 5d22790a6c4c34f65c7ae6cd3823610c7dc7d249
Author: XuQianJin-Stars <fo...@apache.com>
AuthorDate: Thu Dec 15 19:21:47 2022 +0800

    improve checkstyle
---
 README.md              | 8 ++++----
 dev/tencent-release.sh | 4 ++--
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/README.md b/README.md
index 31598a05672..caa77a7ac2c 100644
--- a/README.md
+++ b/README.md
@@ -62,7 +62,7 @@ Prerequisites for building Apache Hudi:
 ```
 # Checkout code and build
 git clone https://github.com/apache/hudi.git && cd hudi
-mvn clean package -DskipTests
+mvn clean package -DskipTests -Dcheckstyle.skip=true -Dscalastyle.skip=true
 
 # Start command
 spark-2.4.4-bin-hadoop2.7/bin/spark-shell \
@@ -97,13 +97,13 @@ Refer to the table below for building with different Spark and Scala versions.
 For example,
 ```
 # Build against Spark 3.2.x
-mvn clean package -DskipTests -Dspark3.2 -Dscala-2.12
+mvn clean package -DskipTests -Dspark3.2 -Dscala-2.12 -Dcheckstyle.skip=true -Dscalastyle.skip=true
 
 # Build against Spark 3.1.x
-mvn clean package -DskipTests -Dspark3.1 -Dscala-2.12
+mvn clean package -DskipTests -Dspark3.1 -Dscala-2.12 -Dcheckstyle.skip=true -Dscalastyle.skip=true
 
 # Build against Spark 2.4.4 and Scala 2.12
-mvn clean package -DskipTests -Dspark2.4 -Dscala-2.12
+mvn clean package -DskipTests -Dspark2.4 -Dscala-2.12 -Dcheckstyle.skip=true -Dscalastyle.skip=true
 ```
 
 #### What about "spark-avro" module?
diff --git a/dev/tencent-release.sh b/dev/tencent-release.sh
index 54631f5c0f1..f8821ea1be9 100644
--- a/dev/tencent-release.sh
+++ b/dev/tencent-release.sh
@@ -117,9 +117,9 @@ function deploy_spark(){
   FLINK_VERSION=$3
 
   if [ ${release_repo} = "Y" ]; then
-    COMMON_OPTIONS="-Dscala-${SCALA_VERSION} -Dspark${SPARK_VERSION} -Dflink${FLINK_VERSION} -DskipTests -s dev/settings.xml -DretryFailedDeploymentCount=30 -T 2.5C"
+    COMMON_OPTIONS="-Dscala-${SCALA_VERSION} -Dspark${SPARK_VERSION} -Dflink${FLINK_VERSION} -DskipTests -Dcheckstyle.skip=true -Dscalastyle.skip=true -s dev/settings.xml -DretryFailedDeploymentCount=30 -T 2.5C"
   else
-    COMMON_OPTIONS="-Dscala-${SCALA_VERSION} -Dspark${SPARK_VERSION} -Dflink${FLINK_VERSION} -DskipTests -s dev/settings.xml -DretryFailedDeploymentCount=30 -T 2.5C"
+    COMMON_OPTIONS="-Dscala-${SCALA_VERSION} -Dspark${SPARK_VERSION} -Dflink${FLINK_VERSION} -DskipTests -Dcheckstyle.skip=true -Dscalastyle.skip=true -s dev/settings.xml -DretryFailedDeploymentCount=30 -T 2.5C"
   fi
 
 #  INSTALL_OPTIONS="-U -Drat.skip=true -Djacoco.skip=true -Dscala-${SCALA_VERSION} -Dspark${SPARK_VERSION} -DskipTests -s dev/settings.xml -T 2.5C"