Skip to content

Commit dcad063

Browse files
author
Bharathy Satish
committed
Merge branch 'mysql-8.0' into mysql-trunk
2 parents 9c42131 + d180878 commit dcad063

File tree

7 files changed

+229
-45
lines changed

7 files changed

+229
-45
lines changed

‎include/mysql/plugin.h

+9-2
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ struct MYSQL_XID {
8989
Plugin API. Common for all plugin types.
9090
*/
9191

92-
#define MYSQL_PLUGIN_INTERFACE_VERSION 0x0108
92+
#define MYSQL_PLUGIN_INTERFACE_VERSION 0x0109
9393

9494
/*
9595
The allowable types of plugins
@@ -178,6 +178,12 @@ __MYSQL_DECLARE_PLUGIN(NAME, \
178178
#define PLUGIN_VAR_MEMALLOC 0x8000 /* String needs memory allocated */
179179
#define PLUGIN_VAR_NOPERSIST 0x10000 /* SET PERSIST_ONLY is prohibited
180180
for read only variables */
181+
/**
182+
There can be some variables which needs to be set before plugin is loaded but
183+
not after plugin is loaded. ex: GR specific variables. Below flag must be set
184+
for these kind of variables.
185+
*/
186+
#define PLUGIN_VAR_PERSIST_AS_READ_ONLY 0x20000
181187

182188
struct SYS_VAR;
183189
struct st_mysql_value;
@@ -231,7 +237,8 @@ typedef void (*mysql_var_update_func)(MYSQL_THD thd,
231237
(PLUGIN_VAR_READONLY | PLUGIN_VAR_NOSYSVAR | \
232238
PLUGIN_VAR_NOCMDOPT | PLUGIN_VAR_NOCMDARG | \
233239
PLUGIN_VAR_OPCMDARG | PLUGIN_VAR_RQCMDARG | PLUGIN_VAR_MEMALLOC | \
234-
PLUGIN_VAR_NODEFAULT | PLUGIN_VAR_NOPERSIST)
240+
PLUGIN_VAR_NODEFAULT | PLUGIN_VAR_NOPERSIST | \
241+
PLUGIN_VAR_PERSIST_AS_READ_ONLY)
235242

236243
#define MYSQL_PLUGIN_VAR_HEADER \
237244
int flags; \

‎rapid/plugin/group_replication/src/plugin.cc

+60-42
Large diffs are not rendered by default.

‎rapid/plugin/group_replication/tests/mtr/r/gr_start_with_persisted_variables.result

+58
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,58 @@
1+
include/group_replication.inc
2+
Warnings:
3+
Note #### Sending passwords in plain text without SSL/TLS is extremely insecure.
4+
Note #### Storing MySQL user name or password information in the master info repository is not secure and is therefore not recommended. Please consider using the USER and PASSWORD connection options for START SLAVE; see the 'START SLAVE Syntax' in the MySQL Manual for more information.
5+
[connection server1]
6+
7+
############################################################
8+
# 1. Persist configuration on server1.
9+
[connection server1]
10+
SET PERSIST group_replication_group_name= "GROUP_REPLICATION_GROUP_NAME";
11+
SET PERSIST group_replication_group_seeds= @@GLOBAL.group_replication_group_seeds;
12+
SET PERSIST group_replication_local_address= @@GLOBAL.group_replication_local_address;
13+
SET PERSIST group_replication_start_on_boot= ON;
14+
SET PERSIST group_replication_bootstrap_group= ON;
15+
16+
############################################################
17+
# 2. Restart server1, it must bootstrap the group.
18+
# restart
19+
include/rpl_reconnect.inc
20+
include/gr_wait_for_member_state.inc
21+
SELECT variable_name, variable_source FROM performance_schema.variables_info WHERE variable_source IN ('PERSISTED');
22+
variable_name variable_source
23+
group_replication_bootstrap_group PERSISTED
24+
group_replication_group_name PERSISTED
25+
group_replication_group_seeds PERSISTED
26+
group_replication_local_address PERSISTED
27+
group_replication_start_on_boot PERSISTED
28+
29+
############################################################
30+
# 3. Persist configuration on server2.
31+
[connection server2]
32+
SET PERSIST group_replication_group_name= "GROUP_REPLICATION_GROUP_NAME";
33+
SET PERSIST group_replication_group_seeds= @@GLOBAL.group_replication_group_seeds;
34+
SET PERSIST group_replication_local_address= @@GLOBAL.group_replication_local_address;
35+
SET PERSIST group_replication_start_on_boot= ON;
36+
37+
############################################################
38+
# 4. Restart server2, it must join the group.
39+
# restart
40+
include/rpl_reconnect.inc
41+
include/gr_wait_for_member_state.inc
42+
include/rpl_gr_wait_for_number_of_members.inc
43+
SELECT variable_name, variable_source FROM performance_schema.variables_info WHERE variable_source IN ('PERSISTED');
44+
variable_name variable_source
45+
group_replication_group_name PERSISTED
46+
group_replication_group_seeds PERSISTED
47+
group_replication_local_address PERSISTED
48+
group_replication_start_on_boot PERSISTED
49+
50+
############################################################
51+
# 5. Clean up.
52+
[connection server1]
53+
RESET PERSIST;
54+
SET GLOBAL group_replication_start_on_boot= OFF;
55+
[connection server2]
56+
RESET PERSIST;
57+
SET GLOBAL group_replication_start_on_boot= OFF;
58+
include/group_replication_end.inc

‎rapid/plugin/group_replication/tests/mtr/t/gr_start_with_persisted_variables.test

+86
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,86 @@
1+
################################################################################
2+
# This test is to test if group replication can be started with needed variables
3+
# persisted into mysqld-auto.cnf file.
4+
# Test:
5+
# 1. Persist configuration on server1.
6+
# 2. Restart server1, it must bootstrap the group.
7+
# 3. Persist configuration on server2.
8+
# 4. Restart server2, it must join the group.
9+
# 5. Clean up.
10+
################################################################################
11+
--source include/big_test.inc
12+
--source ../inc/have_group_replication_plugin.inc
13+
--let $rpl_skip_group_replication_start= 1
14+
--source ../inc/group_replication.inc
15+
16+
17+
--echo
18+
--echo ############################################################
19+
--echo # 1. Persist configuration on server1.
20+
--let $rpl_connection_name= server1
21+
--source include/rpl_connection.inc
22+
--replace_result $group_replication_group_name GROUP_REPLICATION_GROUP_NAME
23+
--eval SET PERSIST group_replication_group_name= "$group_replication_group_name"
24+
--eval SET PERSIST group_replication_group_seeds= @@GLOBAL.group_replication_group_seeds
25+
--eval SET PERSIST group_replication_local_address= @@GLOBAL.group_replication_local_address
26+
SET PERSIST group_replication_start_on_boot= ON;
27+
SET PERSIST group_replication_bootstrap_group= ON;
28+
29+
30+
--echo
31+
--echo ############################################################
32+
--echo # 2. Restart server1, it must bootstrap the group.
33+
--let $allow_rpl_inited= 1
34+
--source include/restart_mysqld.inc
35+
--let $rpl_server_number= 1
36+
--source include/rpl_reconnect.inc
37+
38+
--let $group_replication_member_state= ONLINE
39+
--source ../inc/gr_wait_for_member_state.inc
40+
41+
SELECT variable_name, variable_source FROM performance_schema.variables_info WHERE variable_source IN ('PERSISTED');
42+
43+
44+
--echo
45+
--echo ############################################################
46+
--echo # 3. Persist configuration on server2.
47+
--let $rpl_connection_name= server2
48+
--source include/rpl_connection.inc
49+
--replace_result $group_replication_group_name GROUP_REPLICATION_GROUP_NAME
50+
--eval SET PERSIST group_replication_group_name= "$group_replication_group_name"
51+
--eval SET PERSIST group_replication_group_seeds= @@GLOBAL.group_replication_group_seeds
52+
--eval SET PERSIST group_replication_local_address= @@GLOBAL.group_replication_local_address
53+
SET PERSIST group_replication_start_on_boot= ON;
54+
55+
56+
--echo
57+
--echo ############################################################
58+
--echo # 4. Restart server2, it must join the group.
59+
--let $allow_rpl_inited= 1
60+
--source include/restart_mysqld.inc
61+
--let $rpl_server_number= 2
62+
--source include/rpl_reconnect.inc
63+
64+
--let $group_replication_member_state= ONLINE
65+
--source ../inc/gr_wait_for_member_state.inc
66+
67+
--let $group_replication_number_of_members= 2
68+
--source ../inc/gr_wait_for_number_of_members.inc
69+
70+
SELECT variable_name, variable_source FROM performance_schema.variables_info WHERE variable_source IN ('PERSISTED');
71+
72+
73+
--echo
74+
--echo ############################################################
75+
--echo # 5. Clean up.
76+
--let $rpl_connection_name= server1
77+
--source include/rpl_connection.inc
78+
RESET PERSIST;
79+
SET GLOBAL group_replication_start_on_boot= OFF;
80+
81+
--let $rpl_connection_name= server2
82+
--source include/rpl_connection.inc
83+
RESET PERSIST;
84+
SET GLOBAL group_replication_start_on_boot= OFF;
85+
86+
--source ../inc/group_replication_end.inc

‎sql/persisted_variable.cc

+2-1
Original file line numberDiff line numberDiff line change
@@ -253,7 +253,8 @@ void Persisted_variables_cache::set_variable(THD *thd, set_var *setvar)
253253
/* modification to in-memory must be thread safe */
254254
mysql_mutex_lock(&m_LOCK_persist_variables);
255255
/* if present update variable with new value else insert into hash */
256-
if (setvar->type == OPT_PERSIST_ONLY && setvar->var->is_readonly())
256+
if ((setvar->type == OPT_PERSIST_ONLY && setvar->var->is_readonly()) ||
257+
setvar->var->is_plugin_var_read_only())
257258
m_persist_ro_variables[tmp_var.key]= tmp_var.value;
258259
else
259260
{

‎sql/set_var.h

+5
Original file line numberDiff line numberDiff line change
@@ -192,6 +192,11 @@ class sys_var
192192
timestamp = 0;
193193
}
194194
virtual bool is_non_persistent() {return flags & NOTPERSIST; }
195+
/**
196+
Check if plugin variable is persisted as a read only variable. For
197+
server variables always return false.
198+
*/
199+
virtual bool is_plugin_var_read_only() { return 0; }
195200

196201
/**
197202
Update the system variable with the default value from either

‎sql/sql_plugin_var.h

+9
Original file line numberDiff line numberDiff line change
@@ -277,6 +277,15 @@ class sys_var_pluginvar: public sys_var
277277
{ return (plugin_var->flags & PLUGIN_VAR_NOPERSIST); }
278278
void set_is_plugin(bool val)
279279
{ is_plugin= val; }
280+
/**
281+
Check if plugin variable is persisted as a read only variable.
282+
283+
@return
284+
@retval true
285+
@retval false
286+
*/
287+
bool is_plugin_var_read_only()
288+
{ return (plugin_var->flags & PLUGIN_VAR_PERSIST_AS_READ_ONLY); }
280289
};
281290

282291
/*

0 commit comments

Comments
 (0)