phoenix-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From sama...@apache.org
Subject phoenix git commit: PHOENIX-4153 Temporarily disable java doc warning check in QA builds
Date Sat, 02 Sep 2017 05:43:57 GMT
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 <samarth@apache.org>
Authored: Fri Sep 1 22:43:46 2017 -0700
Committer: Samarth Jain <samarth@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 ""


Mime
View raw message