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

Add support for compressed connection from ProxySQL to backend servers #363

Closed
2 tasks
aismail opened this issue Aug 26, 2015 · 3 comments
Closed
2 tasks

Comments

@aismail
Copy link
Collaborator

aismail commented Aug 26, 2015

Why

Depending on the traffic patterns, sometimes it makes sense to enable compressed connection to the backend servers.

What

  • mysql_servers.compression is taken into account as the min_compress_length, the smallest package size for which compression will be enabled
  • backend connection code enables the compression options for the MariaDB client library when needed

Notes

Will create a follow-up issue for automated Docker test to see that the compression is indeed working. The idea for that is to check the amount of traffic on the network interface used by ProxySQL to communicate with the backend server before and after sending a big package with compression enabled. That big package should be easy to compress - something like REPEAT('a', 1048576).

@renecannao
Copy link
Contributor

Variable min_compress_length needs to be deprecated , and mysql_servers.compression should become a "boolean"

@aismail
Copy link
Collaborator Author

aismail commented Aug 26, 2015

@renecannao why do you want to deprecate it? I recognize the use-case for it was quite subtle :)

@renecannao
Copy link
Contributor

Because it is not configurable in the client library. It is actually hardcoded :
#define MIN_COMPRESS_LENGTH 50

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants