[SPARK-24526][build][test-maven] Spaces in the build dir causes failures in the build/mvn script

[trystanleftwich]

What changes were proposed in this pull request?

Fix the call to ${MVN_BIN} to be wrapped in quotes so it will handle having spaces in the path.

How was this patch tested?

Ran the following to confirm using the build/mvn tool with a space in the build dir now works without error

mkdir /tmp/test\ spaces
cd /tmp/test\ spaces
git clone https://github.com/apache/spark.git
cd spark
# Remove all mvn references in PATH so the script will download mvn to the local dir
./build/mvn -DskipTests clean package

Please review http://spark.apache.org/contributing.html before opening a pull request.

[HyukjinKwon]

ok to test

[SparkQA]

Test build #91696 has finished for PR 21534 at commit bb12f3e.

• This patch fails Spark unit tests.
• This patch merges cleanly.
• This patch adds no public classes.

[mgaido91]

retest this please

[SparkQA]

Test build #91709 has finished for PR 21534 at commit bb12f3e.

• This patch fails Spark unit tests.
• This patch merges cleanly.
• This patch adds no public classes.

[HyukjinKwon]

retest this please

[SparkQA]

Test build #91975 has finished for PR 21534 at commit bb12f3e.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds no public classes.

[HyukjinKwon]

LGTM

[HyukjinKwon]

@trystanleftwich, mind if I ask fix the title to [SPARK-24526][build][test-maven] Spaces in the build dir causes failures in the build/mvn script so that Jenkins triggers a build with Maven and then we can see if the build still passes for sure?

[trystanleftwich]

@HyukjinKwon Updated the title per your request.

[HyukjinKwon]

retest this please

[SparkQA]

Test build #91998 has finished for PR 21534 at commit bb12f3e.

• This patch fails Spark unit tests.
• This patch merges cleanly.
• This patch adds no public classes.

[HyukjinKwon]

retest this please

[SparkQA]

Test build #92002 has finished for PR 21534 at commit bb12f3e.

• This patch fails Spark unit tests.
• This patch merges cleanly.
• This patch adds no public classes.

[HyukjinKwon]

retest this please

[SparkQA]

Test build #92009 has finished for PR 21534 at commit bb12f3e.

• This patch fails Spark unit tests.
• This patch merges cleanly.
• This patch adds no public classes.

[HyukjinKwon]

Merged to master. It ran mvn correctly anyway.