Repository: spark
Updated Branches:
  refs/heads/branch-2.4 5bc4e7d1a -> af2ec972d


[SPARK-25930][K8S] Fix scala string detection in k8s tests

## What changes were proposed in this pull request?

- Issue is described in detail in 
[SPARK-25930](https://issues.apache.org/jira/browse/SPARK-25930). Since we rely 
on the std output, pick always the last line which contains the wanted value. 
Although minor, current implementation breaks tests.

## How was this patch tested?
manually. rm -rf ~/.m2 and then run the tests.

Closes #22931 from skonto/fix_scala_detection.

Authored-by: Stavros Kontopoulos <[email protected]>
Signed-off-by: Sean Owen <[email protected]>
(cherry picked from commit 1fb3759f2b60a2e7c5e2a82afe1a580d848e0f8c)
Signed-off-by: Sean Owen <[email protected]>


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

Branch: refs/heads/branch-2.4
Commit: af2ec972d7022fac369e7b6125f04134983486cf
Parents: 5bc4e7d
Author: Stavros Kontopoulos <[email protected]>
Authored: Mon Nov 5 08:40:25 2018 -0600
Committer: Sean Owen <[email protected]>
Committed: Mon Nov 5 08:40:41 2018 -0600

----------------------------------------------------------------------
 .../integration-tests/dev/dev-run-integration-tests.sh        | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/spark/blob/af2ec972/resource-managers/kubernetes/integration-tests/dev/dev-run-integration-tests.sh
----------------------------------------------------------------------
diff --git 
a/resource-managers/kubernetes/integration-tests/dev/dev-run-integration-tests.sh
 
b/resource-managers/kubernetes/integration-tests/dev/dev-run-integration-tests.sh
index cb5cf69..1b9de48 100755
--- 
a/resource-managers/kubernetes/integration-tests/dev/dev-run-integration-tests.sh
+++ 
b/resource-managers/kubernetes/integration-tests/dev/dev-run-integration-tests.sh
@@ -28,7 +28,12 @@ NAMESPACE=
 SERVICE_ACCOUNT=
 INCLUDE_TAGS="k8s"
 EXCLUDE_TAGS=
-SCALA_VERSION="$($TEST_ROOT_DIR/build/mvn 
org.apache.maven.plugins:maven-help-plugin:2.1.1:evaluate 
-Dexpression=scala.binary.version | grep -v '\[' )"
+MVN="$TEST_ROOT_DIR/build/mvn"
+
+SCALA_VERSION=$("$MVN" help:evaluate -Dexpression=scala.binary.version 
2>/dev/null\
+    | grep -v "INFO"\
+    | grep -v "WARNING"\
+    | tail -n 1)
 
 # Parse arguments
 while (( "$#" )); do


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to