From e65854c2c8bbeaf1a5799f25c52a47ed1d87e0a7 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Javier=20Jaramago=20Fern=C3=A1ndez?= Date: Tue, 28 Mar 2023 14:06:06 +0200 Subject: [PATCH] Remove incorrectly ported checks over checksums 'diff_check' from previous impl --- lib/ProxySQL_Cluster.cpp | 20 ++++++++------------ 1 file changed, 8 insertions(+), 12 deletions(-) diff --git a/lib/ProxySQL_Cluster.cpp b/lib/ProxySQL_Cluster.cpp index 752e5310a..6a2e5d16c 100644 --- a/lib/ProxySQL_Cluster.cpp +++ b/lib/ProxySQL_Cluster.cpp @@ -455,12 +455,10 @@ void ProxySQL_Node_Entry::set_checksums(MYSQL_RES *_r) { hostname, port, checksums_values.admin_variables.version, checksums_values.admin_variables.epoch, checksums_values.admin_variables.checksum ); } else { - if (checksums_values.admin_variables.diff_check == 0) { - proxy_info( - "Cluster: detected a new checksum for admin_variables from peer %s:%d, version %llu, epoch %llu, checksum %s. Not syncing due to 'admin-checksum_admin_variables=false'.\n", - hostname, port, checksums_values.admin_variables.version, checksums_values.admin_variables.epoch, checksums_values.admin_variables.checksum - ); - } + proxy_info( + "Cluster: detected a new checksum for admin_variables from peer %s:%d, version %llu, epoch %llu, checksum %s. Not syncing due to 'admin-checksum_admin_variables=false'.\n", + hostname, port, checksums_values.admin_variables.version, checksums_values.admin_variables.epoch, checksums_values.admin_variables.checksum + ); } } else { checksums_values.admin_variables.diff_check++; @@ -494,12 +492,10 @@ void ProxySQL_Node_Entry::set_checksums(MYSQL_RES *_r) { hostname, port, checksums_values.mysql_query_rules.version, checksums_values.mysql_query_rules.epoch, checksums_values.mysql_query_rules.checksum ); } else { - if (checksums_values.mysql_query_rules.diff_check == 0) { - proxy_info( - "Cluster: detected a new checksum for mysql_query_rules from peer %s:%d, version %llu, epoch %llu, checksum %s. Not syncing due to 'admin-checksum_mysql_query_rules=false'.\n", - hostname, port, checksums_values.mysql_query_rules.version, checksums_values.mysql_query_rules.epoch, checksums_values.mysql_query_rules.checksum - ); - } + proxy_info( + "Cluster: detected a new checksum for mysql_query_rules from peer %s:%d, version %llu, epoch %llu, checksum %s. Not syncing due to 'admin-checksum_mysql_query_rules=false'.\n", + hostname, port, checksums_values.mysql_query_rules.version, checksums_values.mysql_query_rules.epoch, checksums_values.mysql_query_rules.checksum + ); } } else { checksums_values.mysql_query_rules.diff_check++;