[KYUUBI #6599] Disable Javadoc/Scaladoc by default
# 🔍 Description This PR disables Javadoc/Scaladoc by default, while still enabling that on Nexus deploy and CI, to speed up the normal building by developers daily performed. ## Types of changes 🔖 - [ ] Bugfix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to change) ## Test Plan 🧪 Apple M1 Max, Java 17 ``` $ time build/mvn clean install -DskipTests ``` before: 1145.67s user 62.41s system 276% cpu 7:16.56 total after: 530.33s user 35.82s system 214% cpu 4:23.69 total --- # Checklist 📝 - [x] This patch was not authored or co-authored using [Generative Tooling](https://www.apache.org/legal/generative-tooling.html) **Be nice. Be informative.** Closes #6599 from pan3793/javadoc. Closes #6599 660f8dc02 [Cheng Pan] Disable Javadoc/Scaladoc by default Authored-by: Cheng Pan <chengpan@apache.org> Signed-off-by: liangbowen <liangbowen@gf.com.cn>
This commit is contained in:
parent
2043d1c0a2
commit
71f95696e5
2
.github/workflows/master.yml
vendored
2
.github/workflows/master.yml
vendored
@ -106,7 +106,7 @@ jobs:
|
||||
- name: Build and test Kyuubi and Spark with maven w/o linters
|
||||
run: |
|
||||
if [[ "${{ matrix.java }}" == "8" && "${{ matrix.spark }}" == "3.5" && "${{ matrix.spark-archive }}" == "" ]]; then
|
||||
MVN_OPT="${MVN_OPT} -Pcodecov"
|
||||
MVN_OPT="${MVN_OPT} -Pcodecov -Dmaven.javadoc.skip=false -Dmaven.scaladoc.skip=false"
|
||||
fi
|
||||
TEST_MODULES="dev/kyuubi-codecov"
|
||||
./build/mvn clean install ${MVN_OPT} -pl ${TEST_MODULES} -am \
|
||||
|
||||
@ -215,7 +215,7 @@ else
|
||||
echo "Making distribution for Kyuubi $VERSION in '$DISTDIR'..."
|
||||
fi
|
||||
|
||||
MVN_DIST_OPT="-DskipTests -Dmaven.javadoc.skip=true -Dmaven.scaladoc.skip=true -Dmaven.source.skip"
|
||||
MVN_DIST_OPT="-DskipTests -Dmaven.source.skip"
|
||||
|
||||
if [[ "$ENABLE_WEBUI" == "true" ]]; then
|
||||
MVN_DIST_OPT="$MVN_DIST_OPT -Pweb-ui"
|
||||
|
||||
9
pom.xml
9
pom.xml
@ -246,7 +246,8 @@
|
||||
we should skip upgrading until MSOURCES-141 gets fixed. -->
|
||||
<maven.plugin.source.version>3.2.1</maven.plugin.source.version>
|
||||
|
||||
<maven.scaladoc.skip>false</maven.scaladoc.skip>
|
||||
<maven.javadoc.skip>true</maven.javadoc.skip>
|
||||
<maven.scaladoc.skip>true</maven.scaladoc.skip>
|
||||
<maven.scalastyle.skip>false</maven.scalastyle.skip>
|
||||
<!-- Needed for consistent times -->
|
||||
<maven.build.timestamp.format>yyyy-MM-dd HH:mm:ss z</maven.build.timestamp.format>
|
||||
@ -2139,8 +2140,6 @@
|
||||
<id>fast</id>
|
||||
<properties>
|
||||
<enforcer.skip>true</enforcer.skip>
|
||||
<maven.javadoc.skip>true</maven.javadoc.skip>
|
||||
<maven.scaladoc.skip>true</maven.scaladoc.skip>
|
||||
<maven.scalastyle.skip>true</maven.scalastyle.skip>
|
||||
<rat.skip>true</rat.skip>
|
||||
<skipTests>true</skipTests>
|
||||
@ -2180,6 +2179,10 @@
|
||||
|
||||
<profile>
|
||||
<id>apache-release</id>
|
||||
<properties>
|
||||
<maven.javadoc.skip>false</maven.javadoc.skip>
|
||||
<maven.scaladoc.skip>false</maven.scaladoc.skip>
|
||||
</properties>
|
||||
<build>
|
||||
<plugins>
|
||||
<!-- Prevent the source-release-assembly execution defined in the Apache parent POM
|
||||
|
||||
Loading…
Reference in New Issue
Block a user