From 2dfd7d35ac307476847525e28f973f35cdddd538 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Markus=20M=C3=A4kel=C3=A4?= Date: Tue, 2 Apr 2019 17:22:26 +0300 Subject: [PATCH] MXS-2418: Crash on trx replay when log_info is enabled If the session starts with no master but later one becomes available, when a transaction is started the code would unconditionally use the master's name in a log message. --- server/modules/routing/readwritesplit/rwsplit_route_stmt.cc | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/server/modules/routing/readwritesplit/rwsplit_route_stmt.cc b/server/modules/routing/readwritesplit/rwsplit_route_stmt.cc index 05e94e66f..a9418d9f3 100644 --- a/server/modules/routing/readwritesplit/rwsplit_route_stmt.cc +++ b/server/modules/routing/readwritesplit/rwsplit_route_stmt.cc @@ -289,9 +289,7 @@ bool RWSplitSession::route_single_stmt(GWBUF* querybuf) if (!succp && should_migrate_trx(target)) { - MXS_INFO("Starting transaction migration from '%s' to '%s'", - m_current_master->name(), - target->name()); + MXS_INFO("Migrating transaction to '%s'", target->name()); /** * Stash the current query so that the transaction replay treats