Skip to content

Conversation

@ericfchang
Copy link
Contributor

What changes were proposed in this pull request?

Spotbugs maven plugin was a recently added plugin before 2.4.0 snapshot artifacts were broken. To ensure it does not affect the maven deploy plugin, this change removes it.

How was this patch tested?

Local build was ran, but this patch will be actually tested by monitoring the apache repo artifacts and making sure metadata is correctly uploaded after this job is ran: https://amplab.cs.berkeley.edu/jenkins/view/Spark%20Packaging/job/spark-master-maven-snapshots/

@liancheng
Copy link
Contributor

add to whitelist

@liancheng
Copy link
Contributor

test this please

@yhuai
Copy link
Contributor

yhuai commented Jul 24, 2018

cc @HyukjinKwon @kiszk

I will merge this PR once it passes the test.

@SparkQA
Copy link

SparkQA commented Jul 24, 2018

Test build #93511 has finished for PR 21865 at commit af0ecf5.

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

@yhuai
Copy link
Contributor

yhuai commented Jul 24, 2018

lgtm. I am merging this PR to master branch. Then, I will kick off https://amplab.cs.berkeley.edu/jenkins/view/Spark%20Packaging/job/spark-master-maven-snapshots/.

@asfgit asfgit closed this in fc21f19 Jul 24, 2018
@HyukjinKwon
Copy link
Member

Thank you all. I couldn't foresee this problem.

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