Skip to content

Add MySQL compression level variable and change default to compression level 3 (v3.0)#4764

Merged
renecannao merged 12 commits intov3.0from
v3.0_compression
Jan 22, 2025
Merged

Add MySQL compression level variable and change default to compression level 3 (v3.0)#4764
renecannao merged 12 commits intov3.0from
v3.0_compression

Conversation

@yashwantsahu20
Copy link
Contributor

Default compression level was 6 for MySQL clients and this was making ProxySQL slower.

Following changes has been made:

  1. Made default compression level as 3.
  2. Added MySQL compression level variable

For detail of issue, #4721

@yashwantsahu20 yashwantsahu20 changed the title [WIP] Added MySQL compression level variable Added MySQL compression level variable Nov 29, 2024
@yashwantsahu20 yashwantsahu20 changed the title Added MySQL compression level variable Add MySQL compression level variable and change default to compression level 3 Nov 29, 2024
Copy link
Collaborator

@JavierJF JavierJF left a comment

Choose a reason for hiding this comment

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

Comments and changes are focused on the TAP test, there are also some suggestions. Thanks!

1.  Moved monotonic_time to util
2.  Now calculate_query_execution_time will return -1 incase of error.
3.  Using util function for setting and getting compression variable values.
4.  Added mysql_query_rules
2. Changed performance diff to float in tap test
@yashwantsahu20 yashwantsahu20 changed the title Add MySQL compression level variable and change default to compression level 3 Add MySQL compression level variable and change default to compression level 3 (v3.0) Jan 16, 2025
@renecannao
Copy link
Contributor

Can one of the admins verify this patch?

@renecannao renecannao merged commit 643b62d into v3.0 Jan 22, 2025
5 of 6 checks passed
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

Successfully merging this pull request may close these issues.

3 participants