You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@phoenix.apache.org by sa...@apache.org on 2017/09/02 05:43:57 UTC
phoenix git commit: PHOENIX-4153 Temporarily disable java doc warning
check in QA builds
Repository: phoenix
Updated Branches:
refs/heads/master 1aabbfa0d -> 81c025b37
PHOENIX-4153 Temporarily disable java doc warning check in QA builds
Project: http://git-wip-us.apache.org/repos/asf/phoenix/repo
Commit: http://git-wip-us.apache.org/repos/asf/phoenix/commit/81c025b3
Tree: http://git-wip-us.apache.org/repos/asf/phoenix/tree/81c025b3
Diff: http://git-wip-us.apache.org/repos/asf/phoenix/diff/81c025b3
Branch: refs/heads/master
Commit: 81c025b37109059cc7d00d1bb558b8e794b60723
Parents: 1aabbfa
Author: Samarth Jain <sa...@apache.org>
Authored: Fri Sep 1 22:43:46 2017 -0700
Committer: Samarth Jain <sa...@apache.org>
Committed: Fri Sep 1 22:43:46 2017 -0700
----------------------------------------------------------------------
dev/test-patch.sh | 58 +++++++++++++++++++++++++-------------------------
1 file changed, 29 insertions(+), 29 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/phoenix/blob/81c025b3/dev/test-patch.sh
----------------------------------------------------------------------
diff --git a/dev/test-patch.sh b/dev/test-patch.sh
index 4454e13..b2c0fa1 100755
--- a/dev/test-patch.sh
+++ b/dev/test-patch.sh
@@ -574,39 +574,39 @@ checkInterfaceAudience () {
###############################################################################
### Check there are no javadoc warnings
-checkJavadocWarnings () {
- echo ""
- echo ""
- echo "======================================================================"
- echo "======================================================================"
- echo " Determining number of patched javadoc warnings."
- echo "======================================================================"
- echo "======================================================================"
- echo ""
- echo ""
- echo "$MVN clean package javadoc:javadoc -DskipTests -D${PROJECT_NAME}PatchProcess > $PATCH_DIR/patchJavadocWarnings.txt 2>&1"
- export MAVEN_OPTS="${MAVEN_OPTS}"
- $MVN clean package javadoc:javadoc -DskipTests -D${PROJECT_NAME}PatchProcess > $PATCH_DIR/patchJavadocWarnings.txt 2>&1
- javadocWarnings=`$GREP '\[WARNING\]' $PATCH_DIR/patchJavadocWarnings.txt | $AWK '/Javadoc Warnings/,EOF' | $GREP warning | $AWK 'BEGIN {total = 0} {total += 1} END {print total}'`
- echo ""
- echo ""
- echo "There appear to be $javadocWarnings javadoc warnings generated by the patched build."
+#checkJavadocWarnings () {
+# echo ""
+# echo ""
+# echo "======================================================================"
+# echo "======================================================================"
+# echo " Determining number of patched javadoc warnings."
+# echo "======================================================================"
+# echo "======================================================================"
+# echo ""
+# echo ""
+# echo "$MVN clean package javadoc:javadoc -DskipTests -D${PROJECT_NAME}PatchProcess > $PATCH_DIR/patchJavadocWarnings.txt 2>&1"
+# export MAVEN_OPTS="${MAVEN_OPTS}"
+# $MVN clean package javadoc:javadoc -DskipTests -D${PROJECT_NAME}PatchProcess > $PATCH_DIR/patchJavadocWarnings.txt 2>&1
+# javadocWarnings=`$GREP '\[WARNING\]' $PATCH_DIR/patchJavadocWarnings.txt | $AWK '/Javadoc Warnings/,EOF' | $GREP warning | $AWK 'BEGIN {total = 0} {total += 1} END {print total}'`
+# echo ""
+# echo ""
+# echo "There appear to be $javadocWarnings javadoc warnings generated by the patched build."
### if current warnings greater than OK_JAVADOC_WARNINGS
- if [[ $javadocWarnings -gt $OK_JAVADOC_WARNINGS ]] ; then
- JIRA_COMMENT="$JIRA_COMMENT
+# if [[ $javadocWarnings -gt $OK_JAVADOC_WARNINGS ]] ; then
+# JIRA_COMMENT="$JIRA_COMMENT
- {color:red}-1 javadoc{color}. The javadoc tool appears to have generated `expr $(($javadocWarnings-$OK_JAVADOC_WARNINGS))` warning messages."
+# {color:red}-1 javadoc{color}. The javadoc tool appears to have generated `expr $(($javadocWarnings-$OK_JAVADOC_WARNINGS))` warning messages."
# Add javadoc output url
- JIRA_COMMENT_FOOTER="Javadoc warnings: $BUILD_URL/artifact/patchprocess/patchJavadocWarnings.txt
-$JIRA_COMMENT_FOOTER"
- return 1
- fi
- JIRA_COMMENT="$JIRA_COMMENT
-
- {color:green}+1 javadoc{color}. The javadoc tool did not generate any warning messages."
- return 0
-}
+# JIRA_COMMENT_FOOTER="Javadoc warnings: $BUILD_URL/artifact/patchprocess/patchJavadocWarnings.txt
+#$JIRA_COMMENT_FOOTER"
+# return 1
+# fi
+# JIRA_COMMENT="$JIRA_COMMENT
+
+# {color:green}+1 javadoc{color}. The javadoc tool did not generate any warning messages."
+# return 0
+#}
checkBuildWithHadoopVersions() {
echo ""