From 2a65066b350c8354b09d3e57b8fdac644020faee Mon Sep 17 00:00:00 2001 From: Sean Busbey Date: Mon, 26 Feb 2018 11:36:44 -0600 Subject: [PATCH] HBASE-20070 refactor website generation * rely on git plumbing commands when checking if we've built the site for a particular commit already * switch to forcing '-e' for bash * add command line switches for: path to hbase, working directory, and publishing * only export JAVA/MAVEN HOME if they aren't already set. * add some docs about assumptions * Update javadoc plugin to consistently be version 3.0.0 * avoid duplicative site invocations on reactor modules * update use of cp command so it works both on linux and mac * manually skip enforcer plugin during build * still doing install of all jars due to MJAVADOC-490, but then skip rebuilding during aggregate reports. * avoid the pager on git-diff by teeing to a log file, which also helps later reviewing in the case of big changesets. Signed-off-by: Michael Stack Signed-off-by: Misty Stanley-Jones --- .../jenkins-scripts/generate-hbase-website.sh | 257 +++++++++++++----- hbase-annotations/pom.xml | 7 + hbase-assembly/pom.xml | 7 - hbase-backup/pom.xml | 7 - hbase-build-configuration/pom.xml | 7 + hbase-client/pom.xml | 7 - hbase-common/pom.xml | 7 - hbase-endpoint/pom.xml | 7 - hbase-examples/pom.xml | 7 - hbase-external-blockcache/pom.xml | 7 - hbase-hadoop-compat/pom.xml | 7 - hbase-hadoop2-compat/pom.xml | 7 - hbase-http/pom.xml | 7 - hbase-it/pom.xml | 7 - hbase-mapreduce/pom.xml | 7 - hbase-metrics-api/pom.xml | 7 - hbase-metrics/pom.xml | 7 - hbase-procedure/pom.xml | 7 - hbase-protocol-shaded/pom.xml | 7 - hbase-protocol/pom.xml | 7 - hbase-replication/pom.xml | 7 - hbase-resource-bundle/pom.xml | 7 - hbase-rest/pom.xml | 7 - hbase-rsgroup/pom.xml | 7 - hbase-server/pom.xml | 7 - .../hbase-shaded-check-invariants/pom.xml | 7 + hbase-shaded/pom.xml | 7 - hbase-shell/pom.xml | 7 - hbase-spark-it/pom.xml | 7 - hbase-testing-util/pom.xml | 11 - hbase-thrift/pom.xml | 7 - hbase-zookeeper/pom.xml | 7 - pom.xml | 15 +- 33 files changed, 213 insertions(+), 280 deletions(-) diff --git a/dev-support/jenkins-scripts/generate-hbase-website.sh b/dev-support/jenkins-scripts/generate-hbase-website.sh index 06d160a64c2..0d7af75d265 100644 --- a/dev-support/jenkins-scripts/generate-hbase-website.sh +++ b/dev-support/jenkins-scripts/generate-hbase-website.sh @@ -23,74 +23,176 @@ # # It needs to be built on a Jenkins server with the label git-websites # -# It expects to have the hbase repo cloned to the directory hbase +# Allows specifying options for working directory, maven repo, and publishing to git +# run with --help for usage. # # If there is a build error, the Jenkins job is configured to send an email -LOCAL_REPO=${WORKSPACE}/.m2/repo -# Nuke the local maven repo each time, to start with a known environment -rm -Rf "${LOCAL_REPO}" -mkdir -p "${LOCAL_REPO}" +declare CURRENT_HBASE_COMMIT +declare PUSHED +declare FILE +declare WEBSITE_COMMIT_MSG +declare -a FILES_TO_REMOVE -# Clean any leftover files in case we are reusing the workspace -rm -Rf -- *.patch *.patch.zip hbase/target target *.txt hbase-site +set -e +function usage { + echo "Usage: ${0} [options] /path/to/hbase/checkout" + echo "" + echo " --working-dir /path/to/use Path for writing logs and a local checkout of hbase-site repo." + echo " if given must exist." + echo " defaults to making a directory in /tmp." + echo " --local-repo /path/for/maven/.m2 Path for putting local maven repo." + echo " if given must exist." + echo " defaults to making a clean directory in --working-dir." + echo " --publish if given, will attempt to push results back to the hbase-site repo." + echo " --help show this usage message." + exit 1 +} +# if no args specified, show usage +if [ $# -lt 1 ]; then + usage +fi + +# Get arguments +declare component_dir +declare working_dir +declare local_repo +declare publish +while [ $# -gt 0 ] +do + case "$1" in + --working-dir) shift; working_dir=$1; shift;; + --local-repo) shift; local_repo=$1; shift;; + --publish) shift; publish="true";; + --) shift; break;; + -*) usage ;; + *) break;; # terminate while loop + esac +done + +# should still have where component checkout is. +if [ $# -lt 1 ]; then + usage +fi +component_dir="$(cd "$(dirname "$1")"; pwd)/$(basename "$1")" + +if [ -z "${working_dir}" ]; then + echo "[DEBUG] defaulting to creating a directory in /tmp" + working_dir=/tmp + while [[ -e ${working_dir} ]]; do + working_dir=/tmp/hbase-generate-website-${RANDOM}.${RANDOM} + done + mkdir "${working_dir}" +else + # absolutes please + working_dir="$(cd "$(dirname "${working_dir}")"; pwd)/$(basename "${working_dir}")" + if [ ! -d "${working_dir}" ]; then + echo "passed working directory '${working_dir}' must already exist." + exit 1 + fi +fi + +echo "You'll find logs and temp files in ${working_dir}" + +if [ -z "${local_repo}" ]; then + echo "[DEBUG] defaulting to creating a local repo within '${working_dir}'" + local_repo="${working_dir}/.m2/repo" + # Nuke the local maven repo each time, to start with a known environment + rm -Rf "${local_repo}" + mkdir -p "${local_repo}" +else + # absolutes please + local_repo="$(cd "$(dirname "${local_repo}")"; pwd)/$(basename "${local_repo}")" + if [ ! -d "${local_repo}" ]; then + echo "passed directory for storing the maven repo '${local_repo}' must already exist." + exit 1 + fi +fi # Set up the environment -export JAVA_HOME=$JDK_1_8_LATEST__HOME -export PATH=$JAVA_HOME/bin:$MAVEN_3_3_3_HOME/bin:$PATH -export MAVEN_OPTS="-XX:MaxPermSize=256m -Dmaven.repo.local=${LOCAL_REPO}" +if [ -z "${JAVA_HOME}" ]; then + JAVA_HOME="${JDK_1_8_LATEST__HOME}" + export JAVA_HOME + export PATH="${JAVA_HOME}/bin:${PATH}" +fi +if [ -z "${MAVEN_HOME}" ]; then + MAVEN_HOME="${MAVEN_3_3_3_HOME}" + export MAVEN_HOME + export PATH="${MAVEN_HOME}/bin:${PATH}" +fi +export MAVEN_OPTS="${MAVEN_OPTS} -Dmaven.repo.local=${local_repo}" # Verify the Maven version mvn -version +# Verify the git version +git --version + +cd "${working_dir}" + +# Clean any leftover files in case we are reusing the workspace +rm -Rf -- *.patch *.patch.zip target *.txt hbase-site # Save and print the SHA we are building -CURRENT_HBASE_COMMIT="$(git log --pretty=format:%H -n1)" +CURRENT_HBASE_COMMIT="$(cd "${component_dir}" && git show-ref --hash --dereference --verify refs/remotes/origin/HEAD)" +# Fail if it's empty +if [ -z "${CURRENT_HBASE_COMMIT}" ]; then + echo "Got back a blank answer for the current HEAD on the remote hbase repository. failing." + exit 1 +fi echo "Current HBase commit: $CURRENT_HBASE_COMMIT" # Clone the hbase-site repo manually so it doesn't trigger spurious # commits in Jenkins. git clone --depth 1 --branch asf-site https://git-wip-us.apache.org/repos/asf/hbase-site.git -# Figure out the last commit we built the site from, and bail if the build -# still represents the SHA of HBase master -cd "${WORKSPACE}/hbase-site" || exit -1 -git log --pretty=%s | grep ${CURRENT_HBASE_COMMIT} -PUSHED=$? -echo "PUSHED is $PUSHED" +# Figure out if the commit of the hbase repo has already been built and bail if so. +declare -i PUSHED +PUSHED=$(cd hbase-site && git rev-list --grep "${CURRENT_HBASE_COMMIT}" --fixed-strings --count HEAD) +echo "[DEBUG] hash was found in $PUSHED commits for hbase-site repository." -if [ $PUSHED -eq 0 ]; then +if [ "${PUSHED}" -ne 0 ]; then echo "$CURRENT_HBASE_COMMIT is already mentioned in the hbase-site commit log. Not building." exit 0 else - echo "$CURRENT_HBASE_COMMIT is not yet mentioned in the hbase-site commit log. Assuming we don't have it yet. $PUSHED" + echo "$CURRENT_HBASE_COMMIT is not yet mentioned in the hbase-site commit log. Assuming we don't have it yet." fi # Go to the hbase directory so we can build the site -cd "${WORKSPACE}/hbase" || exit -1 +cd "${component_dir}" # This will only be set for builds that are triggered by SCM change, not manual builds -if [ "$CHANGE_ID" ]; then +if [ -n "$CHANGE_ID" ]; then echo -n " ($CHANGE_ID - $CHANGE_TITLE)" fi # Build and install HBase, then build the site echo "Building HBase" -mvn \ - -DskipTests \ - -Dmaven.javadoc.skip=true \ - --batch-mode \ - -Dcheckstyle.skip=true \ - -Dfindbugs.skip=true \ - --log-file="${WORKSPACE}/hbase-build-log-${CURRENT_HBASE_COMMIT}.txt" \ - clean install \ -&& mvn clean site \ - --batch-mode \ - -DskipTests \ - --log-file="${WORKSPACE}/hbase-install-log-${CURRENT_HBASE_COMMIT}.txt" - -status=$? -if [ $status -ne 0 ]; then - echo "Failure: mvn clean site" +# TODO we have to do a local install first because for whatever reason, the maven-javadoc-plugin's +# forked compile phase requires that test-scoped dependencies be available, which +# doesn't work since we will not have done a test-compile phase (MJAVADOC-490). the first place this +# breaks for me is hbase-server trying to find hbase-http:test and hbase-zookeeper:test. +# But! some sunshine: because we're doing a full install before running site, we can skip all the +# compiling in the forked executions. We have to do it awkwardly because MJAVADOC-444. +if mvn \ + -DskipTests \ + -Dmaven.javadoc.skip=true \ + --batch-mode \ + -Denforcer.skip=true \ + -Dcheckstyle.skip=true \ + -Dfindbugs.skip=true \ + --log-file="${working_dir}/hbase-install-log-${CURRENT_HBASE_COMMIT}.txt" \ + clean install \ + && mvn site \ + --batch-mode \ + -Denforcer.skip=true \ + -Dmaven.main.skip=true \ + -Dmaven.test.skip=true \ + -DskipTests \ + --log-file="${working_dir}/hbase-site-log-${CURRENT_HBASE_COMMIT}.txt"; then + echo "Successfully built site." +else + status=$? + echo "Maven commands to build the site failed. check logs for details ${working_dir}/hbase-*-log-*.txt" exit $status fi @@ -98,7 +200,7 @@ fi echo "Staging HBase site" mvn \ --batch-mode \ - --log-file="${WORKSPACE}/hbase-stage-log-${CURRENT_HBASE_COMMIT}.txt" \ + --log-file="${working_dir}/hbase-stage-log-${CURRENT_HBASE_COMMIT}.txt" \ site:stage status=$? if [ $status -ne 0 ] || [ ! -d target/staging ]; then @@ -107,7 +209,7 @@ if [ $status -ne 0 ] || [ ! -d target/staging ]; then fi # Get ready to update the hbase-site repo with the new artifacts -cd "${WORKSPACE}/hbase-site" || exit -1 +cd "${working_dir}/hbase-site" #Remove previously-generated files FILES_TO_REMOVE=("hbase-*" @@ -125,46 +227,59 @@ FILES_TO_REMOVE=("hbase-*" "images") for FILE in "${FILES_TO_REMOVE[@]}"; do - echo "Removing ${WORKSPACE}/hbase-site/$FILE" - rm -Rf "${FILE}" + if [ -e "${FILE}" ]; then + echo "Removing hbase-site/$FILE" + rm -Rf "${FILE}" + fi done # Copy in the newly-built artifacts -cp -au "${WORKSPACE}"/hbase/target/staging/* . +# TODO what do we do when the site build wants to remove something? Can't rsync because e.g. release-specific docs. +cp -pPR "${component_dir}"/target/staging/* . # If the index.html is missing, bail because this is serious if [ ! -f index.html ]; then echo "The index.html is missing. Aborting." exit 1 -else - # Add all the changes - echo "Adding all the files we know about" - git add . - # Create the commit message and commit the changes - WEBSITE_COMMIT_MSG="Published site at $CURRENT_HBASE_COMMIT." - echo "WEBSITE_COMMIT_MSG: $WEBSITE_COMMIT_MSG" - git commit -m "${WEBSITE_COMMIT_MSG}" -a - # Dump a little report - echo "This commit changed these files (excluding Modified files):" - git diff --name-status --diff-filter=ADCRTXUB origin/asf-site - # Create a patch, which Jenkins can save as an artifact and can be examined for debugging - git format-patch --stdout origin/asf-site > "${WORKSPACE}/${CURRENT_HBASE_COMMIT}.patch" - echo "Change set saved to patch ${WORKSPACE}/${CURRENT_HBASE_COMMIT}.patch" - # Push the real commit - git push origin asf-site || (echo "Failed to push to asf-site. Website not updated." && exit -1) - # Create an empty commit to work around INFRA-10751 - git commit --allow-empty -m "INFRA-10751 Empty commit" - # Push the empty commit - git push origin asf-site || (echo "Failed to push the empty commit to asf-site. Website may not update. Manually push an empty commit to fix this. (See INFRA-10751)" && exit -1) - echo "Pushed the changes to branch asf-site. Refresh http://hbase.apache.org/ to see the changes within a few minutes." - git fetch origin - git reset --hard origin/asf-site - - # Zip up the patch so Jenkins can save it - cd "${WORKSPACE}" || exit -1 - zip website.patch.zip "${CURRENT_HBASE_COMMIT}.patch" fi -#echo "Dumping current environment:" -#env +echo "Adding all the files we know about" +git add . +# Create the commit message and commit the changes +WEBSITE_COMMIT_MSG="Published site at $CURRENT_HBASE_COMMIT." +echo "WEBSITE_COMMIT_MSG: $WEBSITE_COMMIT_MSG" +git commit -m "${WEBSITE_COMMIT_MSG}" -a +# Dump a little report +echo "This commit changed these files (excluding Modified files):" +git diff --name-status --diff-filter=ADCRTXUB origin/asf-site | tee "${working_dir}/hbase-file-diff-summary-${CURRENT_HBASE_COMMIT}.txt" +# Create a patch, which Jenkins can save as an artifact and can be examined for debugging +git format-patch --stdout origin/asf-site > "${working_dir}/${CURRENT_HBASE_COMMIT}.patch" +if [ ! -s "${working_dir}/${CURRENT_HBASE_COMMIT}.patch" ]; then + echo "Something went wrong when creating the patch of our updated site." + exit 1 +fi +echo "Change set saved to patch ${working_dir}/${CURRENT_HBASE_COMMIT}.patch" +if [ -n "${publish}" ]; then + echo "Publishing changes to remote repo..." + if git push origin asf-site; then + "changes pushed." + else + echo "Failed to push to asf-site. Website not updated." + exit 1 + fi + echo "Sending empty commit to work around INFRA-10751." + git commit --allow-empty -m "INFRA-10751 Empty commit" + # Push the empty commit + if git push origin asf-site; then + echo "empty commit pushed." + else + echo "Failed to push the empty commit to asf-site. Website may not update. Manually push an empty commit to fix this. (See INFRA-10751)" + exit 1 + fi + echo "Pushed the changes to branch asf-site. Refresh http://hbase.apache.org/ to see the changes within a few minutes." +fi + +# Zip up the patch so Jenkins can save it +cd "${working_dir}" +zip website.patch.zip "${CURRENT_HBASE_COMMIT}.patch" diff --git a/hbase-annotations/pom.xml b/hbase-annotations/pom.xml index 94c506b99ca..a4aa7a1cb63 100644 --- a/hbase-annotations/pom.xml +++ b/hbase-annotations/pom.xml @@ -33,6 +33,13 @@ + + org.apache.maven.plugins + maven-site-plugin + + true + + org.apache.maven.plugins maven-checkstyle-plugin diff --git a/hbase-assembly/pom.xml b/hbase-assembly/pom.xml index 75cba4dc794..8d1cc777c5e 100644 --- a/hbase-assembly/pom.xml +++ b/hbase-assembly/pom.xml @@ -98,13 +98,6 @@ - - org.apache.maven.plugins - maven-site-plugin - - true - - maven-dependency-plugin diff --git a/hbase-backup/pom.xml b/hbase-backup/pom.xml index 3cf3c39fefa..a0c544538d2 100644 --- a/hbase-backup/pom.xml +++ b/hbase-backup/pom.xml @@ -31,13 +31,6 @@ Backup for HBase - - org.apache.maven.plugins - maven-site-plugin - - true - - maven-assembly-plugin diff --git a/hbase-build-configuration/pom.xml b/hbase-build-configuration/pom.xml index 058c95cc313..ac98044335b 100644 --- a/hbase-build-configuration/pom.xml +++ b/hbase-build-configuration/pom.xml @@ -34,6 +34,13 @@ pom + + org.apache.maven.plugins + maven-site-plugin + + true + + maven-assembly-plugin diff --git a/hbase-client/pom.xml b/hbase-client/pom.xml index 459e47d699b..dffa892050e 100644 --- a/hbase-client/pom.xml +++ b/hbase-client/pom.xml @@ -35,13 +35,6 @@ - - org.apache.maven.plugins - maven-site-plugin - - true - - maven-assembly-plugin diff --git a/hbase-common/pom.xml b/hbase-common/pom.xml index 4220adf1f8e..a97bf75d619 100644 --- a/hbase-common/pom.xml +++ b/hbase-common/pom.xml @@ -62,13 +62,6 @@ org.apache.maven.plugins maven-remote-resources-plugin - - org.apache.maven.plugins - maven-site-plugin - - true - - maven-assembly-plugin diff --git a/hbase-endpoint/pom.xml b/hbase-endpoint/pom.xml index d24ea7bd9db..3c23b1fa748 100644 --- a/hbase-endpoint/pom.xml +++ b/hbase-endpoint/pom.xml @@ -35,13 +35,6 @@ - - org.apache.maven.plugins - maven-site-plugin - - true - - org.apache.maven.plugins diff --git a/hbase-examples/pom.xml b/hbase-examples/pom.xml index e86d4f1850b..757dd8431db 100644 --- a/hbase-examples/pom.xml +++ b/hbase-examples/pom.xml @@ -32,13 +32,6 @@ - - org.apache.maven.plugins - maven-site-plugin - - true - - maven-assembly-plugin diff --git a/hbase-external-blockcache/pom.xml b/hbase-external-blockcache/pom.xml index 630650fbb37..13f53dc9a25 100644 --- a/hbase-external-blockcache/pom.xml +++ b/hbase-external-blockcache/pom.xml @@ -46,13 +46,6 @@ org.apache.maven.plugins maven-remote-resources-plugin - - org.apache.maven.plugins - maven-site-plugin - - true - - maven-assembly-plugin diff --git a/hbase-hadoop-compat/pom.xml b/hbase-hadoop-compat/pom.xml index 5d7daa32722..0237e08096f 100644 --- a/hbase-hadoop-compat/pom.xml +++ b/hbase-hadoop-compat/pom.xml @@ -37,13 +37,6 @@ - - org.apache.maven.plugins - maven-site-plugin - - true - - maven-assembly-plugin diff --git a/hbase-hadoop2-compat/pom.xml b/hbase-hadoop2-compat/pom.xml index 4c9cc6a49f4..9b06293e40e 100644 --- a/hbase-hadoop2-compat/pom.xml +++ b/hbase-hadoop2-compat/pom.xml @@ -35,13 +35,6 @@ limitations under the License. - - org.apache.maven.plugins - maven-site-plugin - - true - - org.apache.maven.plugins diff --git a/hbase-http/pom.xml b/hbase-http/pom.xml index 84f098a6e30..9e726c23b8e 100644 --- a/hbase-http/pom.xml +++ b/hbase-http/pom.xml @@ -51,13 +51,6 @@ - - org.apache.maven.plugins - maven-site-plugin - - true - - org.apache.maven.plugins diff --git a/hbase-it/pom.xml b/hbase-it/pom.xml index f75024de6d3..805b6dfa921 100644 --- a/hbase-it/pom.xml +++ b/hbase-it/pom.xml @@ -53,13 +53,6 @@ - - org.apache.maven.plugins - maven-site-plugin - - true - - org.apache.maven.plugins diff --git a/hbase-mapreduce/pom.xml b/hbase-mapreduce/pom.xml index 76dd01571b9..c34516c9b02 100644 --- a/hbase-mapreduce/pom.xml +++ b/hbase-mapreduce/pom.xml @@ -38,13 +38,6 @@ - - org.apache.maven.plugins - maven-site-plugin - - true - - maven-assembly-plugin diff --git a/hbase-metrics-api/pom.xml b/hbase-metrics-api/pom.xml index d9de343320a..1c8a482dc73 100644 --- a/hbase-metrics-api/pom.xml +++ b/hbase-metrics-api/pom.xml @@ -33,13 +33,6 @@ - - org.apache.maven.plugins - maven-site-plugin - - true - - org.apache.maven.plugins diff --git a/hbase-metrics/pom.xml b/hbase-metrics/pom.xml index 4daee1c958b..062ac6836cd 100644 --- a/hbase-metrics/pom.xml +++ b/hbase-metrics/pom.xml @@ -33,13 +33,6 @@ - - org.apache.maven.plugins - maven-site-plugin - - true - - org.apache.maven.plugins diff --git a/hbase-procedure/pom.xml b/hbase-procedure/pom.xml index c791002e724..b94df8ff585 100644 --- a/hbase-procedure/pom.xml +++ b/hbase-procedure/pom.xml @@ -34,13 +34,6 @@ - - org.apache.maven.plugins - maven-site-plugin - - true - - org.apache.maven.plugins diff --git a/hbase-protocol-shaded/pom.xml b/hbase-protocol-shaded/pom.xml index 07ae2d91402..6517b3dee3a 100644 --- a/hbase-protocol-shaded/pom.xml +++ b/hbase-protocol-shaded/pom.xml @@ -47,13 +47,6 @@ - - org.apache.maven.plugins - maven-site-plugin - - true - - org.apache.maven.plugins diff --git a/hbase-protocol/pom.xml b/hbase-protocol/pom.xml index 57c789cf3e3..1d1814b6fe4 100644 --- a/hbase-protocol/pom.xml +++ b/hbase-protocol/pom.xml @@ -34,13 +34,6 @@ - - org.apache.maven.plugins - maven-site-plugin - - true - - org.apache.maven.plugins diff --git a/hbase-replication/pom.xml b/hbase-replication/pom.xml index 86a844a6313..c04d65ecf44 100644 --- a/hbase-replication/pom.xml +++ b/hbase-replication/pom.xml @@ -33,13 +33,6 @@ - - org.apache.maven.plugins - maven-site-plugin - - true - - maven-assembly-plugin diff --git a/hbase-resource-bundle/pom.xml b/hbase-resource-bundle/pom.xml index 275008f0696..efde939c010 100644 --- a/hbase-resource-bundle/pom.xml +++ b/hbase-resource-bundle/pom.xml @@ -39,13 +39,6 @@ - - org.apache.maven.plugins - maven-site-plugin - - true - - maven-assembly-plugin diff --git a/hbase-rest/pom.xml b/hbase-rest/pom.xml index b9c50a0d350..c182f650677 100644 --- a/hbase-rest/pom.xml +++ b/hbase-rest/pom.xml @@ -50,13 +50,6 @@ - - org.apache.maven.plugins - maven-site-plugin - - true - - maven-assembly-plugin diff --git a/hbase-rsgroup/pom.xml b/hbase-rsgroup/pom.xml index 0cde4c7ad69..d56f2506c22 100644 --- a/hbase-rsgroup/pom.xml +++ b/hbase-rsgroup/pom.xml @@ -31,13 +31,6 @@ Regionserver Groups for HBase - - org.apache.maven.plugins - maven-site-plugin - - true - - maven-assembly-plugin diff --git a/hbase-server/pom.xml b/hbase-server/pom.xml index d2cf916035e..950e97f19eb 100644 --- a/hbase-server/pom.xml +++ b/hbase-server/pom.xml @@ -65,13 +65,6 @@ - - org.apache.maven.plugins - maven-site-plugin - - true - - org.apache.maven.plugins diff --git a/hbase-shaded/hbase-shaded-check-invariants/pom.xml b/hbase-shaded/hbase-shaded-check-invariants/pom.xml index d8bf7d2a290..7322769f0b0 100644 --- a/hbase-shaded/hbase-shaded-check-invariants/pom.xml +++ b/hbase-shaded/hbase-shaded-check-invariants/pom.xml @@ -69,6 +69,13 @@ + + org.apache.maven.plugins + maven-site-plugin + + true + + org.apache.maven.plugins maven-enforcer-plugin diff --git a/hbase-shaded/pom.xml b/hbase-shaded/pom.xml index 856d6c3c12c..24c515844e4 100644 --- a/hbase-shaded/pom.xml +++ b/hbase-shaded/pom.xml @@ -64,13 +64,6 @@ - - org.apache.maven.plugins - maven-site-plugin - - true - - maven-assembly-plugin diff --git a/hbase-shell/pom.xml b/hbase-shell/pom.xml index deb2fe4b7f7..58ef7ed444a 100644 --- a/hbase-shell/pom.xml +++ b/hbase-shell/pom.xml @@ -50,13 +50,6 @@ - - org.apache.maven.plugins - maven-site-plugin - - true - - diff --git a/hbase-spark-it/pom.xml b/hbase-spark-it/pom.xml index d1bab947374..c1c39679844 100644 --- a/hbase-spark-it/pom.xml +++ b/hbase-spark-it/pom.xml @@ -46,13 +46,6 @@ - - org.apache.maven.plugins - maven-site-plugin - - true - - org.apache.maven.plugins diff --git a/hbase-testing-util/pom.xml b/hbase-testing-util/pom.xml index d819a6d9a13..0f1b86cf83c 100644 --- a/hbase-testing-util/pom.xml +++ b/hbase-testing-util/pom.xml @@ -29,17 +29,6 @@ hbase-testing-util Apache HBase - Testing Util HBase Testing Utilities. - - - - org.apache.maven.plugins - maven-site-plugin - - true - - - - diff --git a/hbase-thrift/pom.xml b/hbase-thrift/pom.xml index be8023bc0e6..a9919532c25 100644 --- a/hbase-thrift/pom.xml +++ b/hbase-thrift/pom.xml @@ -69,13 +69,6 @@ - - org.apache.maven.plugins - maven-site-plugin - - true - - maven-assembly-plugin diff --git a/hbase-zookeeper/pom.xml b/hbase-zookeeper/pom.xml index 8fb5b15144a..d4a47e2fa2b 100644 --- a/hbase-zookeeper/pom.xml +++ b/hbase-zookeeper/pom.xml @@ -51,13 +51,6 @@ - - org.apache.maven.plugins - maven-site-plugin - - true - - diff --git a/pom.xml b/pom.xml index db8a78882b3..99415cdc30e 100755 --- a/pom.xml +++ b/pom.xml @@ -606,6 +606,7 @@ org.apache.maven.plugins maven-javadoc-plugin + ${maven.javadoc.version} org.apache.maven.plugins @@ -1474,6 +1475,7 @@ 2.10 2.5.2 3.0.2 + 3.0.0 1.2 2.15.2 3.0.0 @@ -1544,6 +1546,9 @@ this parameter by invoking mvn with -Dbuild.id=$BUILD_ID--> ${maven.build.timestamp} bash + + none @@ -2239,14 +2244,6 @@ - - org.apache.maven.plugins - maven-javadoc-plugin - - - -Xdoclint:none - - org.codehaus.mojo findbugs-maven-plugin @@ -3486,7 +3483,7 @@ org.apache.maven.plugins maven-javadoc-plugin - 3.0.0 + ${maven.javadoc.version}