Skip to content

Commit 30300c7

Browse files
committed
Add different compression choice
Summary: Test Plan: Reviewers: Subscribers: Tasks: Tags:
1 parent e1491ff commit 30300c7

File tree

8 files changed

+100
-0
lines changed

8 files changed

+100
-0
lines changed

include/mysql/components/services/clone_protocol_service.h

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -61,6 +61,8 @@ struct mysql_clone_ssl_context {
6161

6262
/** Enable network compression. */
6363
bool m_enable_compression;
64+
const char *m_compression_algorithm;
65+
uint m_compression_level;
6466
NET_SERVER *m_server_extn;
6567
};
6668

Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
INSTALL PLUGIN clone SONAME 'CLONE_PLUGIN';
2+
SET GLOBAL clone_enable_compression = ON;
3+
SET GLOBAL clone_compression_algorithm = ZSTD;
4+
SET GLOBAL clone_zstd_compression_level = 8;
5+
SET GLOBAL clone_autotune_concurrency = OFF;
6+
SET GLOBAL clone_max_concurrency = 8;
7+
SET GLOBAL clone_valid_donor_list = 'HOST:PORT';
8+
CLONE INSTANCE FROM USER@HOST:PORT IDENTIFIED BY '' DATA DIRECTORY = 'CLONE_DATADIR';
9+
select ID, STATE, ERROR_NO from performance_schema.clone_status;
10+
ID STATE ERROR_NO
11+
1 Completed 0
12+
select ID, STAGE, STATE from performance_schema.clone_progress;
13+
ID STAGE STATE
14+
1 DROP DATA Completed
15+
1 FILE COPY Completed
16+
1 PAGE COPY Completed
17+
1 SST COPY Completed
18+
1 REDO COPY Completed
19+
1 FILE SYNC Completed
20+
1 RESTART Not Started
21+
1 RECOVERY Not Started
22+
# restart
23+
UNINSTALL PLUGIN clone;
Lines changed: 28 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,28 @@
1+
--source include/have_rocksdb.inc
2+
3+
--let $HOST = 127.0.0.1
4+
--let $PORT =`select @@port`
5+
--let $USER = root
6+
--let remote_clone = 1
7+
8+
--source ../../clone/include/clone_connection_begin.inc
9+
--let $CLONE_DATADIR = $MYSQL_TMP_DIR/data_new
10+
11+
# Install Clone Plugin
12+
--replace_result $CLONE_PLUGIN CLONE_PLUGIN
13+
--eval INSTALL PLUGIN clone SONAME '$CLONE_PLUGIN'
14+
15+
--connection clone_conn_1
16+
SET GLOBAL clone_enable_compression = ON;
17+
SET GLOBAL clone_compression_algorithm = ZSTD;
18+
SET GLOBAL clone_zstd_compression_level = 8;
19+
--source ../../clone/include/clone_command.inc
20+
21+
--force-rmdir $CLONE_DATADIR
22+
23+
--let restart_parameters=
24+
--source include/restart_mysqld.inc
25+
26+
# Clean up
27+
UNINSTALL PLUGIN clone;
28+
--source ../../clone/include/clone_connection_end.inc

plugin/clone/include/clone.h

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -90,6 +90,10 @@ extern uint clone_max_io_bandwidth;
9090
/** Clone system variable: If network compression is enabled */
9191
extern bool clone_enable_compression;
9292

93+
extern unsigned long clone_compression_algorithm;
94+
95+
extern uint clone_zstd_compression_level;
96+
9397
/** Clone system variable: SSL private key */
9498
extern char *clone_client_ssl_private_key;
9599

plugin/clone/include/clone_client.h

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,10 @@ using Time_Msec = std::chrono::milliseconds;
5151
using Time_Sec = std::chrono::seconds;
5252
using Time_Min = std::chrono::minutes;
5353

54+
static const char *clone_compression_algorithms[] = {
55+
"dummy", "UNCOMPRESSED", "ZLIB", "ZSTD",
56+
"ZSTD_STREAM", "LZ4F_STREAM", "INVALID", NullS};
57+
5458
struct Thread_Info {
5559
/** Default constructor */
5660
Thread_Info() = default;

plugin/clone/src/clone_client.cc

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,10 @@ Clone Plugin: Client implementation
2626
2727
*/
2828
#include <inttypes.h>
29+
#include <string>
2930

31+
#include "my_compress.h"
32+
#include "plugin/clone/include/clone.h"
3033
#include "plugin/clone/include/clone_client.h"
3134
#include "plugin/clone/include/clone_os.h"
3235

@@ -896,6 +899,13 @@ int Client::connect_remote(bool is_restart, bool use_aux) {
896899
mysql_clone_ssl_context ssl_context;
897900

898901
ssl_context.m_enable_compression = clone_enable_compression;
902+
903+
if (ssl_context.m_enable_compression) {
904+
ssl_context.m_compression_algorithm =
905+
clone_compression_algorithms[clone_compression_algorithm];
906+
ssl_context.m_compression_level = clone_zstd_compression_level;
907+
}
908+
899909
ssl_context.m_server_extn =
900910
ssl_context.m_enable_compression ? &m_conn_server_extn : nullptr;
901911
ssl_context.m_ssl_mode = m_share->m_ssl_mode;

plugin/clone/src/clone_plugin.cc

Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -70,6 +70,10 @@ uint clone_max_io_bandwidth;
7070
/** Clone system variable: If network compression is enabled */
7171
bool clone_enable_compression;
7272

73+
unsigned long clone_compression_algorithm;
74+
75+
uint clone_zstd_compression_level;
76+
7377
/** Clone system variable: valid list of donor addresses. */
7478
static char *clone_valid_donor_list;
7579

@@ -636,6 +640,24 @@ static MYSQL_SYSVAR_BOOL(enable_compression, clone_enable_compression,
636640
"If compression is done at network", nullptr, nullptr,
637641
false); /* Disable compression by default */
638642

643+
static TYPELIB clone_compression_algorithms_typelib = {
644+
array_elements(myclone::clone_compression_algorithms) - 1,
645+
"clone_compression_algorithms_typelib",
646+
myclone::clone_compression_algorithms, nullptr};
647+
648+
static MYSQL_SYSVAR_ENUM(compression_algorithm, clone_compression_algorithm,
649+
PLUGIN_VAR_OPCMDARG | PLUGIN_VAR_MEMALLOC,
650+
"compression algorithm used in clone", nullptr,
651+
nullptr, enum_compression_algorithm::MYSQL_ZSTD,
652+
&clone_compression_algorithms_typelib);
653+
654+
static MYSQL_SYSVAR_UINT(zstd_compression_level, clone_zstd_compression_level,
655+
PLUGIN_VAR_NOCMDARG, "zstd compression level", nullptr,
656+
nullptr, 3, /* Default */
657+
1, /* Minimum */
658+
10, /* Maximum */
659+
1);
660+
639661
/** List of valid donor addresses allowed to clone from. */
640662
static MYSQL_SYSVAR_STR(valid_donor_list, clone_valid_donor_list,
641663
PLUGIN_VAR_OPCMDARG | PLUGIN_VAR_MEMALLOC,
@@ -705,6 +727,8 @@ static SYS_VAR *clone_system_variables[] = {
705727
MYSQL_SYSVAR(ssl_cert),
706728
MYSQL_SYSVAR(ssl_ca),
707729
MYSQL_SYSVAR(donor_timeout_after_network_failure),
730+
MYSQL_SYSVAR(compression_algorithm),
731+
MYSQL_SYSVAR(zstd_compression_level),
708732
nullptr};
709733

710734
/** Declare clone plugin */

sql/server_component/clone_protocol_service.cc

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -394,6 +394,11 @@ DEFINE_METHOD(MYSQL *, mysql_clone_connect,
394394
/* Enable compression. */
395395
if (ssl_ctx->m_enable_compression) {
396396
mysql_options(mysql, MYSQL_OPT_COMPRESS, nullptr);
397+
if (ssl_ctx->m_compression_algorithm)
398+
mysql_options(mysql, MYSQL_OPT_COMPRESSION_ALGORITHMS,
399+
ssl_ctx->m_compression_algorithm);
400+
mysql_options(mysql, MYSQL_OPT_ZSTD_COMPRESSION_LEVEL,
401+
&(ssl_ctx->m_compression_level));
397402
mysql_extension_set_server_extn(mysql, ssl_ctx->m_server_extn);
398403
}
399404

0 commit comments

Comments
 (0)