Skip to content

Conversation

@sarutak
Copy link
Member

@sarutak sarutak commented Jan 11, 2016

#10659 removed the repository https://repo.eclipse.org/content/repositories/paho-releases but it's needed by MiMa because spark-streaming-mqtt(1.6.0) depends on mqttv3(1.0.1) and it is provided by the removed repository and maven-central provide only mqttv3(1.0.2) for now.

Otherwise, if mqttv3(1.0.1) is absent from the local repository, dev/mima should fail.

@JoshRosen Do you have any other better idea?

@sarutak
Copy link
Member Author

sarutak commented Jan 11, 2016

The MiMa failure in #10685, #10683 and #10678 may be related this issue.

@SparkQA
Copy link

SparkQA commented Jan 11, 2016

Test build #49079 has finished for PR 10688 at commit b354d49.

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

Copy link
Contributor

Choose a reason for hiding this comment

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

I wonder if you have to add this to defaultResolvers instead in order for it to take effect? Also, you might want to += rather than := to avoid completely overwriting the existing resolvers, right?

@sarutak sarutak force-pushed the SPARK-4628-followup branch from 1397c43 to c173399 Compare January 11, 2016 02:33
@SparkQA
Copy link

SparkQA commented Jan 11, 2016

Test build #49083 has finished for PR 10688 at commit 1397c43.

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

@SparkQA
Copy link

SparkQA commented Jan 11, 2016

Test build #49084 has finished for PR 10688 at commit c173399.

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

@rxin
Copy link
Contributor

rxin commented Jan 11, 2016

cc @JoshRosen I'm going to merge this so other pull requests don't fail.

@asfgit asfgit closed this in 008a558 Jan 11, 2016
@srowen
Copy link
Member

srowen commented Jan 11, 2016

@sarutak @rxin why not just depend on version 1.0.2 then? sounds like a simple update

@sarutak sarutak deleted the SPARK-4628-followup branch April 12, 2016 13:59
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