mirror of
				https://github.com/MariaDB/server.git
				synced 2025-10-25 18:38:00 +03:00 
			
		
		
		
	If setting a system-variable provided by a plug-in failed, no OK or error was sent in some cases, hanging the client. We now send an error in the case from the ticket (integer-argument out of range in STRICT mode). We also provide a semi-generic fallback message for possible future cases like this where an error is signalled, but no message is sent to the client. The error/warning handling is unified so it's the same again for variables provided by plugins and those in the server proper. mysql-test/r/plugin.result: show that on out-of-range values, plugin interface throws errors in STRICT mode and warnings otherwise. mysql-test/t/plugin.test: show that on out-of-range values, plugin interface throws errors in STRICT mode and warnings otherwise. sql/set_var.cc: - handle signedness of values used in warnings - in STRICT mode, throw errors rather than warnings sql/sql_parse.cc: If sql_set_variables() returns with an error but no message was sent to the client, send a semi-generic one so the session won't hang and we won't fail silently. sql/sql_plugin.cc: throw a warning if more than just block-size was corrected (or an error in STRICT mode). use functions from set_var for uniform behaviour of server- and plug-in variables. storage/example/ha_example.cc: Add a ULONG system variable to example plugin so we can test integers in the plugin-interface without having to depend on the presence of innobase.
		
			
				
	
	
		
			57 lines
		
	
	
		
			1.8 KiB
		
	
	
	
		
			Plaintext
		
	
	
	
	
	
			
		
		
	
	
			57 lines
		
	
	
		
			1.8 KiB
		
	
	
	
		
			Plaintext
		
	
	
	
	
	
| CREATE TABLE t1(a int) ENGINE=EXAMPLE;
 | |
| Warnings:
 | |
| Warning	1286	Unknown table engine 'EXAMPLE'
 | |
| Warning	1266	Using storage engine MyISAM for table 't1'
 | |
| DROP TABLE t1;
 | |
| INSTALL PLUGIN example SONAME 'ha_example.so';
 | |
| INSTALL PLUGIN EXAMPLE SONAME 'ha_example.so';
 | |
| ERROR HY000: Function 'EXAMPLE' already exists
 | |
| UNINSTALL PLUGIN example;
 | |
| INSTALL PLUGIN example SONAME 'ha_example.so';
 | |
| CREATE TABLE t1(a int) ENGINE=EXAMPLE;
 | |
| SELECT * FROM t1;
 | |
| a
 | |
| DROP TABLE t1;
 | |
| UNINSTALL PLUGIN example;
 | |
| UNINSTALL PLUGIN EXAMPLE;
 | |
| ERROR 42000: PLUGIN EXAMPLE does not exist
 | |
| UNINSTALL PLUGIN non_exist;
 | |
| ERROR 42000: PLUGIN non_exist does not exist
 | |
| #
 | |
| # Bug#32034: check_func_enum() does not check correct values but set it
 | |
| #            to impossible int val
 | |
| #
 | |
| INSTALL PLUGIN example SONAME 'ha_example.so';
 | |
| SET GLOBAL example_enum_var= e1;
 | |
| SET GLOBAL example_enum_var= e2;
 | |
| SET GLOBAL example_enum_var= impossible;
 | |
| ERROR 42000: Variable 'enum_var' can't be set to the value of 'impossible'
 | |
| UNINSTALL PLUGIN example;
 | |
| INSTALL PLUGIN example SONAME 'ha_example.so';
 | |
| select @@session.sql_mode into @old_sql_mode;
 | |
| set session sql_mode='';
 | |
| set global example_ulong_var=500;
 | |
| select @@global.example_ulong_var;
 | |
| @@global.example_ulong_var
 | |
| 500
 | |
| set global example_ulong_var=1111;
 | |
| Warnings:
 | |
| Warning	1292	Truncated incorrect ulong_var value: '1111'
 | |
| select @@global.example_ulong_var;
 | |
| @@global.example_ulong_var
 | |
| 1000
 | |
| set session sql_mode='STRICT_ALL_TABLES';
 | |
| set global example_ulong_var=500;
 | |
| select @@global.example_ulong_var;
 | |
| @@global.example_ulong_var
 | |
| 500
 | |
| set global example_ulong_var=1111;
 | |
| ERROR 42000: Variable 'ulong_var' can't be set to the value of '1111'
 | |
| select @@global.example_ulong_var;
 | |
| @@global.example_ulong_var
 | |
| 500
 | |
| set session sql_mode=@old_sql_mode;
 | |
| set session old=bla;
 | |
| ERROR HY000: Variable 'old' is a read only variable
 | |
| UNINSTALL PLUGIN example;
 |