Skip to content

Implemented Frame Size Negotiation in ALTS for gRPC Java - #6840

Merged
jiangtaoli2016 merged 1 commit into
grpc:masterfrom
ashithasantosh:java_fsn
Apr 10, 2020
Merged

Implemented Frame Size Negotiation in ALTS for gRPC Java#6840
jiangtaoli2016 merged 1 commit into
grpc:masterfrom
ashithasantosh:java_fsn

Conversation

@ashithasantosh

Copy link
Copy Markdown
Contributor

gRPC Java uses zero copy frame protectors, and hence are capable of handling larger frame size. Frame Size Negotiation support in ALTS, extends send frame size from 16KB to 128KB, and thereby improves performance.

Design Document: go/seal-frame-size-negotiation-alts

@jiangtaoli2016 jiangtaoli2016 added the kokoro:force-run Add this label to a PR to tell Kokoro to re-run all tests. Not generally necessary label Mar 18, 2020
@grpc-kokoro grpc-kokoro removed the kokoro:force-run Add this label to a PR to tell Kokoro to re-run all tests. Not generally necessary label Mar 18, 2020
Comment thread alts/src/main/java/io/grpc/alts/internal/AltsHandshakerClient.java Outdated
Comment thread alts/src/main/java/io/grpc/alts/internal/AltsTsiHandshaker.java Outdated

@jiangtaoli2016 jiangtaoli2016 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.

Thanks @ashithasantosh for PR.

@jiangtaoli2016

Copy link
Copy Markdown
Contributor

@ashithasantosh , please work with gRPC folks (e.g., @ejona86 ) on the internal patch, since you removed the function getMaxAllowedFrameBytes.

@ashithasantosh

Copy link
Copy Markdown
Contributor Author

I created a CL that imports this PR.
https://critique.corp.google.com/#review/305926248
This CL includes update to Loas2TsiHandshaker.java

After that CL is reviewed, we can merge this PR and then I will submit that CL.

@ashithasantosh

Copy link
Copy Markdown
Contributor Author

Thanks for the review, @jiangtaoli2016!

@jiangtaoli2016 jiangtaoli2016 added the kokoro:force-run Add this label to a PR to tell Kokoro to re-run all tests. Not generally necessary label Apr 10, 2020
@grpc-kokoro grpc-kokoro removed the kokoro:force-run Add this label to a PR to tell Kokoro to re-run all tests. Not generally necessary label Apr 10, 2020
@jiangtaoli2016
jiangtaoli2016 merged commit cfe73eb into grpc:master Apr 10, 2020
dfawley pushed a commit to dfawley/grpc-java that referenced this pull request Jan 15, 2021
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Jun 15, 2021
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.

3 participants