Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MQTT streaming: target Akka 2.6 #2118

Merged
merged 5 commits into from Feb 13, 2020
Merged

MQTT streaming: target Akka 2.6 #2118

merged 5 commits into from Feb 13, 2020

Conversation

raboof
Copy link
Member

@raboof raboof commented Jan 31, 2020

Still failing one test: "MqttSessionSpec: MQTT client connector should receive a QoS 1 publication from a subscribed topic and ack it and then ack it again"

References #2097

@probot-autolabeler probot-autolabeler bot added dependency-change For PRs changing the version of a dependency. p:mqtt-streaming labels Jan 31, 2020
@ennru
Copy link
Member

ennru commented Jan 31, 2020

Watch out for the Alpakka testkit dependency brought in by alpakkaProject.

@ennru ennru added this to Incoming Issues and PRs in Akka streaming via automation Jan 31, 2020
@ennru ennru moved this from Incoming Issues and PRs to In progress in Akka streaming Jan 31, 2020
@raboof raboof marked this pull request as ready for review January 31, 2020 16:51
@raboof
Copy link
Member Author

raboof commented Jan 31, 2020

(failure is scaladoc generation, looking into that...)

Copy link
Contributor

@huntc huntc left a comment

Choose a reason for hiding this comment

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

Nice

@ennru
Copy link
Member

ennru commented Feb 12, 2020

The failure is not new to the branch #1723

Akka streaming automation moved this from In progress to Reviewer approved Feb 12, 2020
Copy link
Member

@ennru ennru left a comment

Choose a reason for hiding this comment

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

Please update the warning in the docs.
LGTM.

@ennru
Copy link
Member

ennru commented Feb 12, 2020

Ah, I think it should state Akka 2.6 only, instead.

@raboof
Copy link
Member Author

raboof commented Feb 12, 2020

it seems obvious from the dependency table. Do you think we should spell it out further?

DeepinScreenshot_select-area_20200212154646

@ennru
Copy link
Member

ennru commented Feb 12, 2020

Yes, it is an exception to the rule in Alpakka.
See https://doc.akka.io/docs/alpakka/2.0/overview.html#versions

@ennru ennru changed the title Target Akka 2.6 for akka-mqtt-streaming MQTT streaming: target Akka 2.6 Feb 13, 2020
@ennru ennru merged commit 06482bc into master Feb 13, 2020
Akka streaming automation moved this from Reviewer approved to Done Feb 13, 2020
@ennru ennru deleted the mqttStreamingForAkka26 branch February 13, 2020 16:33
@ennru ennru modified the milestone: 2.0.0-M3 Feb 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependency-change For PRs changing the version of a dependency. documentation p:mqtt-streaming
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants