[15/19] yetus git commit: YETUS-373. build driver: maven install should be smarter

[aw...@apache.org]

YETUS-373. build driver: maven install should be smarter

Signed-off-by: Allen Wittenauer <aw...@apache.org>


Project: http://git-wip-us.apache.org/repos/asf/yetus/repo
Commit: http://git-wip-us.apache.org/repos/asf/yetus/commit/2a80bc0f
Tree: http://git-wip-us.apache.org/repos/asf/yetus/tree/2a80bc0f
Diff: http://git-wip-us.apache.org/repos/asf/yetus/diff/2a80bc0f

Branch: refs/heads/YETUS-156
Commit: 2a80bc0f6dab2a1c02dfd9a0ca09b7c5f207f77c
Parents: 2dff168
Author: Allen Wittenauer <aw...@apache.org>
Authored: Tue Apr 12 13:52:33 2016 -0700
Committer: Allen Wittenauer <aw...@apache.org>
Committed: Sun Apr 17 14:07:17 2016 -0700

----------------------------------------------------------------------
 precommit/test-patch.d/maven.sh | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/yetus/blob/2a80bc0f/precommit/test-patch.d/maven.sh
----------------------------------------------------------------------
diff --git a/precommit/test-patch.d/maven.sh b/precommit/test-patch.d/maven.sh
index 77aab32..d65aeaa 100755
--- a/precommit/test-patch.d/maven.sh
+++ b/precommit/test-patch.d/maven.sh
@@ -523,7 +523,14 @@ function maven_precompile
   fi
 
   if verify_needed_test javac; then
-    need=true
+    # if we are in full build mode, then
+    # there is no need to do an initial install
+    # since we will be doing a full compile from basedir
+    # for projects like hadoop, this saves quite a bit
+    # of time
+    if [[ "${BUILDMODE}" = patch ]]; then
+      need=true
+    fi
   else
     # not everything needs a maven install
     # but quite a few do ...
@@ -542,10 +549,6 @@ function maven_precompile
    return 0
   fi
 
-  if [[ "${BUILDMODE}" = Full ]]; then
-    return 0
-  fi
-
   if [[ "${repostatus}" = branch ]]; then
     big_console_header "maven install: ${PATCH_BRANCH}"
   else