Skip to content

Conversation

@HyukjinKwon
Copy link
Member

What changes were proposed in this pull request?

Maven version was upgraded and AppVeyor should also use upgraded maven version.

Currently, it looks broken by this:

https://ci.appveyor.com/project/ApacheSoftwareFoundation/spark/build/2458-master

[WARNING] Rule 0: org.apache.maven.plugins.enforcer.RequireMavenVersion failed with message:
Detected Maven Version: 3.3.9 is not in the allowed range 3.5.4.
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:

How was this patch tested?

AppVeyor tests

@holdensmagicalunicorn
Copy link

@HyukjinKwon, thanks! I am a bot who has found some folks who might be able to help with the review:@shivaram

@kiszk
Copy link
Member

kiszk commented Jul 30, 2018

@HyukjinKwon thank you

@shivaram
Copy link
Contributor

LGTM. Thanks @HyukjinKwon

@SparkQA
Copy link

SparkQA commented Jul 30, 2018

Test build #93795 has finished for PR 21920 at commit 976d2dd.

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

@HyukjinKwon
Copy link
Member Author

Merged to master.

Thank you @kiszk and @shivaram.

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.

5 participants