-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Disable bundleStrings
RPC extension
#11636
Conversation
cc @tortmayr I'm open for any other ideas on how to address this issue. Feel free to suggest something else than disabling the |
Thanks! :Change looks good to me, I can confirm that plugin support now works as expected after switching the language. 👍🏼
I also looked into this issue and for the moment it looks like the only option is to disable the I also extended and rerun the benchmark extension (introduced for #11159): ResultsTestruns: 500
It looks like disabling the I'm going to open an issue for this in the |
@tortmayr Thanks for looking into it and rerunning the benchmarks! Really appreciated :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I confirm the problem on master
and that this change fixes it. 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I confirmed the bug on master, and that the change fixes it 👍
What it does
Closes #11624
Closes #11631
Disables the
bundleStrings
encoding/decoding extension formsgpackr
. Although it aims to improve performance, it leads to decoding errors on large objects (> a few mb).How to test
Configure Display Language
command.Review checklist
Reminder for reviewers