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

feat: increase max receive message size #399

Merged
merged 3 commits into from
Aug 30, 2022

Conversation

jroper
Copy link
Member

@jroper jroper commented Aug 2, 2022

This increases the gRPC maximum receive message size to 12MB, and also makes both it and the max send size configurable.

@github-actions github-actions bot added kalix-runtime Runtime and SDKs sub-team typescript-sdk labels Aug 2, 2022
tck/package.json Outdated
@@ -11,7 +11,8 @@
},
"devDependencies": {
"testcontainers": "7.10.0",
"typescript": "4.3.4"
"typescript": "4.3.4",
"protobufjs": "6.11.3"
Copy link
Member Author

Choose a reason for hiding this comment

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

This was to work around #398 until we have a better solution.

@jroper jroper force-pushed the increase-max-recv-msg-size branch from be936f7 to a58a095 Compare August 3, 2022 09:29
@jroper
Copy link
Member Author

jroper commented Aug 3, 2022

CI appears to be failing because of #398. I worked around that in one place for this PR, but it seems that it's going to happen everywhere. We need to fix #398.

@octonato
Copy link
Member

octonato commented Aug 4, 2022

Depends on #401
#401 will 'fix' the protobufjs issue

@jroper jroper force-pushed the increase-max-recv-msg-size branch from a58a095 to b4ec291 Compare August 4, 2022 10:43
This increases the gRPC maximum receive message size to 12MB, and also
makes both it and the max send size configurable.
@octonato octonato force-pushed the increase-max-recv-msg-size branch from b4ec291 to 1aa0f77 Compare August 4, 2022 14:42
@octonato
Copy link
Member

octonato commented Aug 4, 2022

@jroper, I rebased this PR and did a push force.

The goal is to merge and release it soon.

Copy link
Member

@octonato octonato left a comment

Choose a reason for hiding this comment

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

LGTM

@octonato
Copy link
Member

octonato commented Aug 4, 2022

Failed because of some documentation issue. I won't have time to check this. @jroper, could have a look?

(watch-out, I did a force-push on your branch)

@johanandren
Copy link
Contributor

That wasn't enough, the ChannelSettings interface needs to go in the exports in index.ts as well.

@pvlugter
Copy link
Member

Added ChannelSettings to the exports.

@pvlugter pvlugter merged commit d78f4ee into lightbend:main Aug 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kalix-runtime Runtime and SDKs sub-team typescript-sdk
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants