Skip to content

Conversation

@wangyum
Copy link
Member

@wangyum wangyum commented Apr 27, 2019

What changes were proposed in this pull request?

Update the docs/building-spark.md. Otherwise:

mvn package -DskipTests=true
...
[INFO] --- maven-enforcer-plugin:3.0.0-M2:enforce (enforce-versions) @ spark-parent_2.12 ---
[WARNING] Rule 0: org.apache.maven.plugins.enforcer.RequireMavenVersion failed with message:
Detected Maven Version: 3.6.0 is not in the allowed range 3.6.1.
...
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-enforcer-plugin:3.0.0-M2:enforce (enforce-versions) on project spark-parent_2.12: Some Enforcer rules have failed. Look above for specific messages explaining why the rule failed. -> [Help 1]
[ERROR]
...

How was this patch tested?

Just test https://archive.apache.org/dist/maven/maven-3/3.6.1/binaries/apache-maven-3.6.1-bin.zip is avilable.

@SparkQA
Copy link

SparkQA commented Apr 27, 2019

Test build #104960 has finished for PR 24477 at commit 76a734b.

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

@dongjoon-hyun dongjoon-hyun changed the title [SPARK-27467][FOLLOW-UP][BUILD] Upgrade Maven to 3.6.1 [SPARK-27467][FOLLOW-UP][BUILD] Upgrade Maven to 3.6.1 in AppVeyor Apr 27, 2019
@dongjoon-hyun dongjoon-hyun changed the title [SPARK-27467][FOLLOW-UP][BUILD] Upgrade Maven to 3.6.1 in AppVeyor [SPARK-27467][FOLLOW-UP][BUILD] Upgrade Maven to 3.6.1 in AppVeyor and Doc Apr 27, 2019
Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1, LGTM. Merged to master.
cc @HyukjinKwon

@wangyum wangyum deleted the SPARK-27467 branch April 28, 2019 00:36
@HyukjinKwon
Copy link
Member

Let me revert this as of #24480. Not sure about the root cause, hm.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants