Skip to content

examples: Add pre-serialized-message example#10112

Merged
ejona86 merged 2 commits into
grpc:masterfrom
ejona86:example-preserialized
Aug 15, 2023
Merged

examples: Add pre-serialized-message example#10112
ejona86 merged 2 commits into
grpc:masterfrom
ejona86:example-preserialized

Conversation

@ejona86

@ejona86 ejona86 commented Apr 28, 2023

Copy link
Copy Markdown
Member

This came out of the question #9707, and could be useful to others.

CC @temawi, MOAR!

This came out of the question grpc#9707, and could be useful to others.
@ejona86 ejona86 requested a review from larry-safran April 28, 2023 16:30

@larry-safran larry-safran left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

You should add an entry to examples/README.md in the Other examples section

Comment thread examples/src/main/java/io/grpc/examples/preserialized/PreSerializedServer.java Outdated
Comment thread examples/src/main/java/io/grpc/examples/preserialized/PreSerializedServer.java Outdated
Comment thread examples/src/main/java/io/grpc/examples/preserialized/PreSerializedServer.java Outdated
@ejona86 ejona86 requested a review from larry-safran August 14, 2023 17:56
@ejona86 ejona86 merged commit 849186a into grpc:master Aug 15, 2023
@ejona86 ejona86 deleted the example-preserialized branch August 15, 2023 00:00
larry-safran pushed a commit to larry-safran/grpc-java that referenced this pull request Aug 29, 2023
This came out of the question grpc#9707, and could be useful to others.
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Nov 13, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants