MDEV-36923 : Add warning messages for applier FK failures

janlindstrom · janlindstrom · commit 359551644f4f · 2025-10-15T17:13:49.000+03:00
Add warning messages if applier write set execution
fails because of foreign key constraint error.
Warnings are printed by default and can be disabled using
SET GLOBAL wsrep_mode=APPLIER_DISABLE_FK_WARNINGS.
diff --git a/include/mysql/service_wsrep.h b/include/mysql/service_wsrep.h
@@ -14,7 +14,7 @@ enum Wsrep_service_key_type
 #else
 
 /* Copyright (c) 2015, 2020, MariaDB Corporation Ab
-                 2018 Codership Oy <info@codership.com>
+                 2018, 2025, Codership Oy <info@codership.com>
 
    This program is free software; you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
@@ -96,6 +96,7 @@ extern struct wsrep_service_st {
   void                        (*wsrep_thd_kill_UNLOCK_func)(const MYSQL_THD thd);
   void                        (*wsrep_thd_set_wsrep_PA_unsafe_func)(MYSQL_THD thd);
   uint32                      (*wsrep_get_domain_id_func)();
+  my_bool                     (*wsrep_applier_log_warnings)(const MYSQL_THD thd);
 } *wsrep_service;
 
 #define MYSQL_SERVICE_WSREP_INCLUDED
@@ -146,6 +147,7 @@ extern struct wsrep_service_st {
 #define wsrep_report_bf_lock_wait(T,I) wsrep_service->wsrep_report_bf_lock_wait(T,I)
 #define wsrep_thd_set_PA_unsafe(T) wsrep_service->wsrep_thd_set_PA_unsafe_func(T)
 #define wsrep_get_domain_id(T) wsrep_service->wsrep_get_domain_id_func(T)
+#define wsrep_applier_log_warnings(T) wsrep_service->wsrep_applier_log_warnings_func(T)
 #else
 
 #define MYSQL_SERVICE_WSREP_STATIC_INCLUDED
@@ -256,5 +258,6 @@ extern "C" void wsrep_report_bf_lock_wait(const THD *thd,
 /* declare parallel applying unsafety for the THD */
 extern "C" void wsrep_thd_set_PA_unsafe(MYSQL_THD thd);
 extern "C" uint32 wsrep_get_domain_id();
+extern "C" my_bool wsrep_applier_log_warnings(const MYSQL_THD thd);
 #endif
 #endif /* MYSQL_SERVICE_WSREP_INCLUDED */
diff --git a/mysql-test/suite/galera/r/MW-369.result b/mysql-test/suite/galera/r/MW-369.result
@@ -258,6 +258,8 @@ SET SESSION DEBUG_SYNC = 'now SIGNAL signal.wsrep_apply_cb';
 SET GLOBAL DEBUG_DBUG = "";
 SET DEBUG_SYNC = 'RESET';
 connection node_1;
+# Now there should be 1 WSREP Foreign key constraint warning
+include/assert_grep.inc [Foreign key warning in table]
 connection node_2;
 SELECT * FROM pg;
 f1	f2
@@ -268,3 +270,44 @@ f1	p_id	f2
 1	1	0
 DROP TABLE cg;
 DROP TABLE pg;
+connection node_1;
+CREATE TABLE `pg-` (f1 INTEGER PRIMARY KEY, f2 INTEGER) ENGINE=INNODB;
+CREATE TABLE `cg-` (f1 INTEGER PRIMARY KEY, p_id INTEGER,
+f2 INTEGER,
+CONSTRAINT `fk_1-$$` FOREIGN KEY (p_id) REFERENCES `pg-` (f1))  ;
+INSERT INTO `pg-` VALUES (1, 0);
+INSERT INTO `pg-` VALUES (2, 0);
+connection node_1;
+SET AUTOCOMMIT=ON;
+START TRANSACTION;
+UPDATE `pg-` SET f2 = 1 WHERE f1 = 1;
+connection node_1a;
+SET SESSION wsrep_sync_wait = 0;
+SET GLOBAL DEBUG_DBUG = "d,sync.wsrep_apply_cb";
+connection node_2;
+INSERT INTO `cg-` VALUES (1, 1, 0);
+connection node_1a;
+SET SESSION DEBUG_SYNC = "now WAIT_FOR sync.wsrep_apply_cb_reached";
+connection node_1;
+SET SESSION DEBUG_SYNC = "wsrep_before_certification SIGNAL before_certification_reached WAIT_FOR continue_before_certification";
+COMMIT;
+connection node_1a;
+SET SESSION DEBUG_SYNC = "now WAIT_FOR before_certification_reached";
+SET SESSION DEBUG_SYNC = 'now SIGNAL continue_before_certification';
+SET SESSION DEBUG_SYNC = 'now SIGNAL signal.wsrep_apply_cb';
+SET GLOBAL DEBUG_DBUG = "";
+SET DEBUG_SYNC = 'RESET';
+connection node_1;
+call mtr.add_suppression("WSREP: Foreign key warning");
+# Now there should be 2 WSREP Foreign key constraint warnings
+include/assert_grep.inc [Foreign key warning in table]
+connection node_2;
+SELECT * FROM `pg-`;
+f1	f2
+1	1
+2	0
+SELECT * FROM `cg-`;
+f1	p_id	f2
+1	1	0
+DROP TABLE `cg-`;
+DROP TABLE `pg-`;
diff --git a/mysql-test/suite/galera/r/galera_FK_duplicate_client_insert.result b/mysql-test/suite/galera/r/galera_FK_duplicate_client_insert.result
@@ -380,3 +380,6 @@ truncate user_session;
 set debug_sync = reset;
 connection node_1;
 drop table user_session,user;
+# Now there should be 10 WSREP Foreign key constraint warnings
+include/assert_grep.inc [Foreign key warning in table]
+call mtr.add_suppression("WSREP: Foreign key warning in table");
diff --git a/mysql-test/suite/galera/r/galera_var_wsrep_mode.result b/mysql-test/suite/galera/r/galera_var_wsrep_mode.result
@@ -20,8 +20,8 @@ SET GLOBAL wsrep_mode='A';
 ERROR 42000: Variable 'wsrep_mode' can't be set to the value of 'A'
 SET GLOBAL wsrep_mode=NULL;
 ERROR 42000: Variable 'wsrep_mode' can't be set to the value of 'NULL'
-SET GLOBAL wsrep_mode=128;
-ERROR 42000: Variable 'wsrep_mode' can't be set to the value of '128'
+SET GLOBAL wsrep_mode=65536;
+ERROR 42000: Variable 'wsrep_mode' can't be set to the value of '65536'
 SET GLOBAL wsrep_mode=REQUIRED_PRIMARY_KEY,REPLICATE_MYISAM;
 ERROR 42000: You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near '' at line 1
 SET GLOBAL wsrep_mode=1;
@@ -56,6 +56,10 @@ SET GLOBAL wsrep_mode=REPLICATE_ARIA;
 SELECT @@wsrep_mode;
 @@wsrep_mode
 REPLICATE_ARIA
+SET GLOBAL wsrep_mode=APPLIER_DISABLE_FK_WARNINGS;
+SELECT @@wsrep_mode;
+@@wsrep_mode
+APPLIER_DISABLE_FK_WARNINGS
 SET GLOBAL wsrep_mode=DISALLOW_LOCAL_GTID;
 SELECT @@wsrep_mode;
 @@wsrep_mode
diff --git a/mysql-test/suite/galera/t/MW-369.test b/mysql-test/suite/galera/t/MW-369.test
@@ -22,7 +22,7 @@
 #
 
 --source include/galera_cluster.inc
---source include/have_innodb.inc
+--source include/have_debug.inc
 --source include/have_debug_sync.inc
 
 CREATE TABLE p (f1 INTEGER PRIMARY KEY, f2 INTEGER) ENGINE=INNODB;
@@ -334,9 +334,49 @@ INSERT INTO pg VALUES (2, 0);
 --connection node_1
 --reap
 
+--echo # Now there should be 1 WSREP Foreign key constraint warning
+--let $assert_count = 1
+--let $assert_file = $MYSQLTEST_VARDIR/log/mysqld.1.err
+--let $assert_text = Foreign key warning in table
+--let $assert_select = Foreign key warning in table
+--source include/assert_grep.inc
+
+
 --connection node_2
 SELECT * FROM pg;
 SELECT * FROM cg;
 
 DROP TABLE cg;
 DROP TABLE pg;
+
+--connection node_1
+CREATE TABLE `pg-` (f1 INTEGER PRIMARY KEY, f2 INTEGER) ENGINE=INNODB;
+CREATE TABLE `cg-` (f1 INTEGER PRIMARY KEY, p_id INTEGER,
+                    f2 INTEGER,
+                    CONSTRAINT `fk_1-$$` FOREIGN KEY (p_id) REFERENCES `pg-` (f1))  ;
+
+INSERT INTO `pg-` VALUES (1, 0);
+INSERT INTO `pg-` VALUES (2, 0);
+
+--let mw_369_parent_query = UPDATE `pg-` SET f2 = 1 WHERE f1 = 1
+--let $mw_369_child_query = INSERT INTO `cg-` VALUES (1, 1, 0)
+--source MW-369.inc
+
+# Commit succeeds
+--connection node_1
+--reap
+call mtr.add_suppression("WSREP: Foreign key warning");
+
+--echo # Now there should be 2 WSREP Foreign key constraint warnings
+--let $assert_count = 2
+--let $assert_file = $MYSQLTEST_VARDIR/log/mysqld.1.err
+--let $assert_text = Foreign key warning in table
+--let $assert_select = Foreign key warning in table
+--source include/assert_grep.inc
+
+--connection node_2
+SELECT * FROM `pg-`;
+SELECT * FROM `cg-`;
+
+DROP TABLE `cg-`;
+DROP TABLE `pg-`;
diff --git a/mysql-test/suite/galera/t/galera_FK_duplicate_client_insert.test b/mysql-test/suite/galera/t/galera_FK_duplicate_client_insert.test
@@ -159,3 +159,17 @@ while($counter > 0)
 --connection node_1
 
 drop table user_session,user;
+
+let $wsrep_mode = `SELECT @@GLOBAL.wsrep_mode`;
+
+if ($wsrep_mode != APPLIER_DISABLE_FK_WARNINGS)
+{
+--echo # Now there should be 10 WSREP Foreign key constraint warnings
+--let $assert_count = 10
+--let $assert_file = $MYSQLTEST_VARDIR/log/mysqld.1.err
+--let $assert_text = Foreign key warning in table
+--let $assert_select = Foreign key warning in table
+--source include/assert_grep.inc
+}
+
+call mtr.add_suppression("WSREP: Foreign key warning in table");
diff --git a/mysql-test/suite/galera/t/galera_var_wsrep_mode.test b/mysql-test/suite/galera/t/galera_var_wsrep_mode.test
@@ -22,7 +22,7 @@ SET GLOBAL wsrep_mode='A';
 --error ER_WRONG_VALUE_FOR_VAR
 SET GLOBAL wsrep_mode=NULL;
 --error ER_WRONG_VALUE_FOR_VAR
-SET GLOBAL wsrep_mode=128;
+SET GLOBAL wsrep_mode=65536;
 --error ER_PARSE_ERROR
 SET GLOBAL wsrep_mode=REQUIRED_PRIMARY_KEY,REPLICATE_MYISAM;
 #
@@ -44,8 +44,11 @@ SET GLOBAL wsrep_mode=REPLICATE_MYISAM;
 SELECT @@wsrep_mode;
 SET GLOBAL wsrep_mode=REPLICATE_ARIA;
 SELECT @@wsrep_mode;
+SET GLOBAL wsrep_mode=APPLIER_DISABLE_FK_WARNINGS;
+SELECT @@wsrep_mode;
 SET GLOBAL wsrep_mode=DISALLOW_LOCAL_GTID;
 SELECT @@wsrep_mode;
+
 SET GLOBAL wsrep_mode='STRICT_REPLICATION,BINLOG_ROW_FORMAT_ONLY';
 SELECT @@wsrep_mode;
 SET GLOBAL wsrep_mode='STRICT_REPLICATION,BINLOG_ROW_FORMAT_ONLY,REQUIRED_PRIMARY_KEY,REPLICATE_MYISAM,REPLICATE_ARIA,DISALLOW_LOCAL_GTID';
diff --git a/mysql-test/suite/sys_vars/r/sysvars_wsrep.result b/mysql-test/suite/sys_vars/r/sysvars_wsrep.result
@@ -357,7 +357,7 @@ VARIABLE_COMMENT	Set of WSREP features that are enabled.
 NUMERIC_MIN_VALUE	NULL
 NUMERIC_MAX_VALUE	NULL
 NUMERIC_BLOCK_SIZE	NULL
-ENUM_VALUE_LIST	STRICT_REPLICATION,BINLOG_ROW_FORMAT_ONLY,REQUIRED_PRIMARY_KEY,REPLICATE_MYISAM,REPLICATE_ARIA,DISALLOW_LOCAL_GTID,BF_ABORT_MARIABACKUP
+ENUM_VALUE_LIST	STRICT_REPLICATION,BINLOG_ROW_FORMAT_ONLY,REQUIRED_PRIMARY_KEY,REPLICATE_MYISAM,REPLICATE_ARIA,DISALLOW_LOCAL_GTID,BF_ABORT_MARIABACKUP,APPLIER_DISABLE_FK_WARNINGS
 READ_ONLY	NO
 COMMAND_LINE_ARGUMENT	REQUIRED
 GLOBAL_VALUE_PATH	NULL
diff --git a/sql/service_wsrep.cc b/sql/service_wsrep.cc
@@ -418,3 +418,15 @@ extern "C" my_bool wsrep_thd_is_local_transaction(const THD *thd)
   return (wsrep_thd_is_local(thd) &&
 	  thd->wsrep_cs().transaction().active());
 }
+
+extern "C" my_bool wsrep_applier_log_warnings(const THD *thd)
+{
+  /* Applier will log warning on fk  errors if
+     (1) wsrep_mode !=  WSREP_MODE_APPLIER_DISABLE_FK_WARNINGS AND
+     (2) thd executing is applier
+     (3) there is no error log warning flooding
+  */
+  return (!wsrep_check_mode(WSREP_MODE_APPLIER_DISABLE_FK_WARNINGS) &&
+          thd->wsrep_cs().mode() == wsrep::client_state::m_high_priority &&
+          !wsrep_protect_against_warning_flood(WSREP_APPLIER_FK_ERROR));
+}
diff --git a/sql/sql_plugin_services.inl b/sql/sql_plugin_services.inl
@@ -185,7 +185,8 @@ static struct wsrep_service_st wsrep_handler = {
   wsrep_thd_kill_LOCK,
   wsrep_thd_kill_UNLOCK,
   wsrep_thd_set_PA_unsafe,
-  wsrep_get_domain_id
+  wsrep_get_domain_id,
+  wsrep_applier_log_warnings
 };
 
 static struct thd_specifics_service_st thd_specifics_handler=
diff --git a/sql/sys_vars.cc b/sql/sys_vars.cc
@@ -6287,6 +6287,7 @@ static const char *wsrep_mode_names[]=
   "REPLICATE_ARIA",
   "DISALLOW_LOCAL_GTID",
   "BF_ABORT_MARIABACKUP",
+  "APPLIER_DISABLE_FK_WARNINGS",
   NullS
 };
 static Sys_var_set Sys_wsrep_mode(
diff --git a/sql/wsrep_dummy.cc b/sql/wsrep_dummy.cc
@@ -169,3 +169,6 @@ void wsrep_thd_set_PA_unsafe(THD*)
 
 uint32 wsrep_get_domain_id()
 { return 0;}
+
+my_bool wsrep_applier_log_warnings(const THD*)
+{ return 0;}
diff --git a/sql/wsrep_mysqld.cc b/sql/wsrep_mysqld.cc
@@ -1235,14 +1235,6 @@ bool wsrep_check_mode (enum_wsrep_mode mask)
 //number of limit warnings after which the suppression will be activated
 #define WSREP_WARNING_ACTIVATION_THRESHOLD 10
 
-enum wsrep_warning_type {
-  WSREP_DISABLED = 0,
-  WSREP_REQUIRE_PRIMARY_KEY= 1,
-  WSREP_REQUIRE_INNODB= 2,
-  WSREP_EXPERIMENTAL= 3,
-  WSREP_REQUIRE_MAX=4,
-};
-
 static ulonglong wsrep_warning_start_time=0;
 static bool wsrep_warning_active[WSREP_REQUIRE_MAX+1];
 static ulonglong wsrep_warning_count[WSREP_REQUIRE_MAX+1];
@@ -1277,6 +1269,8 @@ static const char* wsrep_warning_name(const enum wsrep_warning_type type)
     return "WSREP_REQUIRE_PRIMARY_KEY"; break;
   case WSREP_REQUIRE_INNODB:
     return "WSREP_REQUIRE_INNODB"; break;
+  case WSREP_APPLIER_FK_ERROR:
+    return "WSREP_APPLIER_FK_ERROR"; break;
   case WSREP_EXPERIMENTAL:
     return "WSREP_EXPERIMENTAL"; break;
 
@@ -1306,7 +1300,7 @@ static const char* wsrep_warning_name(const enum wsrep_warning_type type)
     1   Message suppressed
 */
 
-static bool wsrep_protect_against_warning_flood(
+bool wsrep_protect_against_warning_flood(
               enum wsrep_warning_type warning_type)
 {
   ulonglong count;
diff --git a/sql/wsrep_mysqld.h b/sql/wsrep_mysqld.h
@@ -1,4 +1,4 @@
-/* Copyright 2008-2023 Codership Oy <http://www.codership.com>
+/* Copyright 2008-2025 Codership Oy <http://www.codership.com>
 
    This program is free software; you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
@@ -131,7 +131,17 @@ enum enum_wsrep_mode {
   WSREP_MODE_REPLICATE_MYISAM= (1ULL << 3),
   WSREP_MODE_REPLICATE_ARIA= (1ULL << 4),
   WSREP_MODE_DISALLOW_LOCAL_GTID= (1ULL << 5),
-  WSREP_MODE_BF_MARIABACKUP= (1ULL << 6)
+  WSREP_MODE_BF_MARIABACKUP= (1ULL << 6),
+  WSREP_MODE_APPLIER_DISABLE_FK_WARNINGS= (1ULL << 7)
+};
+
+enum wsrep_warning_type {
+  WSREP_DISABLED = 0,
+  WSREP_REQUIRE_PRIMARY_KEY= 1,
+  WSREP_REQUIRE_INNODB= 2,
+  WSREP_EXPERIMENTAL= 3,
+  WSREP_APPLIER_FK_ERROR= 4,
+  WSREP_REQUIRE_MAX=5,
 };
 
 // Streaming Replication
@@ -610,6 +620,8 @@ bool wsrep_table_list_has_non_temp_tables(THD *thd, TABLE_LIST *tables);
  */
 bool wsrep_foreign_key_append(THD *thd, FOREIGN_KEY_INFO *fk);
 
+bool wsrep_protect_against_warning_flood(
+       enum wsrep_warning_type warning_type);
 #else /* !WITH_WSREP */
 
 /* These macros are needed to compile MariaDB without WSREP support
diff --git a/storage/innobase/row/row0ins.cc b/storage/innobase/row/row0ins.cc
