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

Google Cloud Pub/Sub gRPC: latest versions #2050

Merged
merged 6 commits into from
Dec 20, 2019
Merged

Conversation

ennru
Copy link
Member

@ennru ennru commented Dec 16, 2019

Purpose

Use the current versions of Akka gRPC and other dependencies.

References

https://github.com/akka/akka-grpc/releases/tag/v0.7.3

@probot-autolabeler probot-autolabeler bot added the dependency-change For PRs changing the version of a dependency. label Dec 16, 2019
@ennru
Copy link
Member Author

ennru commented Dec 17, 2019

Added some instructions in response to #1255

Copy link
Contributor

@raboof raboof left a comment

Choose a reason for hiding this comment

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

mostly looks good

@@ -305,6 +308,7 @@ lazy val docs = project
"extref.akka-http.base_url" -> s"https://doc.akka.io/docs/akka-http/${Dependencies.AkkaHttpBinaryVersion}/%s",
"scaladoc.akka.http.base_url" -> s"https://doc.akka.io/api/akka-http/${Dependencies.AkkaHttpBinaryVersion}/",
"javadoc.akka.http.base_url" -> s"https://doc.akka.io/japi/akka-http/${Dependencies.AkkaHttpBinaryVersion}/",
"extref.akka-grpc.base_url" -> s"https://doc.akka.io/docs/akka-grpc/current/%s",
Copy link
Contributor

Choose a reason for hiding this comment

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

(not sure this is worth it if we're only using it in 2 places, but OK)


### Maven

Configure your project to use the Java agent for ALPN and add `-javaagent:...` to your startup scripts as described in the @extref:[Akka gRPC documentation](akka-grpc:/buildtools/maven.html#starting-your-akka-grpc-server-from-maven).
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this is only needed on JDK's 8 and earlier - we should double-check and update the Akka gRPC docs to make this clear as well.

Copy link
Member Author

Choose a reason for hiding this comment

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

That's right, I've heard that before... please let me know when you confirmed it.

Copy link
Contributor

Choose a reason for hiding this comment

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

I just confirmed it with the Java akka-grpc-quickstart, added akka/akka-grpc#772

@ennru
Copy link
Member Author

ennru commented Dec 20, 2019

Something goes bananas with during unidoc https://travis-ci.com/akka/alpakka/jobs/269243062#L532

@ennru ennru removed the p:cassandra label Dec 20, 2019
@ennru ennru merged commit 28d1f84 into akka:master Dec 20, 2019
@ennru ennru deleted the grpc-updates branch December 20, 2019 16:19
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:google-cloud-pub-sub-grpc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants