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

Add example for gRPC using Maven #1091

Merged
merged 3 commits into from
Jul 8, 2020

Conversation

Scottmitch
Copy link
Member

No description provided.

<url>https://servicetalk.io</url>

<properties>
<servicetalk.version>0.29.0</servicetalk.version>
Copy link
Member Author

Choose a reason for hiding this comment

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

keeping versions up to date maybe painful down the line, but since this is meant to be an example from the user's perspective it is representative of actual use.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Any way we can source gradle.properties and get the version?

Copy link
Member Author

Choose a reason for hiding this comment

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

we maybe able to but because gradle and maven treat build dependencies differently using the version verbatim would lead to unresolved artifacts. I'll update the release script to bump this version.

Copy link
Collaborator

@NiteshKant NiteshKant left a comment

Choose a reason for hiding this comment

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

Adding the example makes sense, few comments then LGTM

import io.grpc.examples.helloworld.HelloReply;
import io.grpc.examples.helloworld.HelloRequest;

public final class BlockingHelloWorldClient {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Also add a server so the example is self sufficient?

Copy link
Member Author

Choose a reason for hiding this comment

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

I just moved the pom.xml into the existing helloworld example to avoid duplication

<url>https://servicetalk.io</url>

<properties>
<servicetalk.version>0.29.0</servicetalk.version>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Any way we can source gradle.properties and get the version?

@Scottmitch Scottmitch merged commit 7fb95d0 into apple:master Jul 8, 2020
@Scottmitch Scottmitch deleted the grpc_maven_example branch July 8, 2020 23:32
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.

2 participants