-
Notifications
You must be signed in to change notification settings - Fork 0
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
[FAB-134] WS Protobuf #56
Conversation
6cbc0a7
to
e2bf36d
Compare
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.
partially done reviewing, managed to run it on localhost so all good.
when profiling before and after using pb, do we want to document it somewhere like in confluence? interested in how u did it and how much performance is gained
The method is nowhere scientific and purely empirical but I'll get it documented |
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Migrate string JSON WS messages to binary Protobuf messages as an attempt to reduce network and CPU load on server and client, thus a better response time and horizontal scaling.