Merge branch '10.2' into bb-10.2-ext

8 files changed, 15 insertions, 46 deletions

diff --git a/mysql-test/include/fail_start_mysqld.inc b/mysql-test/include/fail_start_mysqld.inc
deleted file mode 100644
index bb1085b5dd7..00000000000
--- a/mysql-test/include/fail_start_mysqld.inc
+++ /dev/null
@@ -1,18 +0,0 @@
-# ==== Usage ====
-#
-# [--let $restart_parameters= --innodb-force-recovery=0 --innodb-read-only=1]
-# [--let $mysqld_stub_cmd= $MYSQLD_LAST_CMD]
-# [--let $error_log= $MYSQLTEST_VARDIR/log/mysqld.1.err]
-# --source include/fail_restart_mysqld.inc
-
-# Evaluate the default of $error_log
-if (!$error_log)
-{
-  --let $error_log= $MYSQLTEST_VARDIR/log/mysqld.1.err
-}
-
---error 1
---exec $mysqld_stub_cmd $restart_parameters >> $error_log  2>&1
-
-# As the server is stopped
---disable_reconnect
diff --git a/mysql-test/mysql-test-run.pl b/mysql-test/mysql-test-run.pl
index aae314e8370..88651907d3f 100755
--- a/mysql-test/mysql-test-run.pl
+++ b/mysql-test/mysql-test-run.pl
@@ -5057,9 +5057,9 @@ sub mysqld_start ($$) {
   }
 
 
-  # "Dynamic" version of MYSQLD_CMD is reevaluated with each mysqld_start.
-  # Use it to restart the server at testing a failing server start (e.g
-  # due to incompatible options).
+  # Command line for mysqld started for *this particular test*.
+  # Differs from "generic" MYSQLD_CMD by including all command line
+  # options from *.opt and *.combination files.
   $ENV{'MYSQLD_LAST_CMD'}= "$exe  @$args";
 
   if ( $opt_gdb || $opt_manual_gdb )
diff --git a/mysql-test/r/tc_heuristic_recover.result b/mysql-test/r/tc_heuristic_recover.result
index 8fe3643b174..193a73c77f8 100644
--- a/mysql-test/r/tc_heuristic_recover.result
+++ b/mysql-test/r/tc_heuristic_recover.result
@@ -1,7 +1,6 @@
 call mtr.add_suppression("Can't init tc log");
 call mtr.add_suppression("Found 1 prepared transactions!");
 call mtr.add_suppression("Aborting");
-set debug_sync='RESET';
 CREATE TABLE t1 (i INT) ENGINE=InnoDB;
 SET GLOBAL innodb_flush_log_at_trx_commit=1;
 FLUSH TABLES;
diff --git a/mysql-test/t/tc_heuristic_recover.test b/mysql-test/t/tc_heuristic_recover.test
index c9fa767c249..8cbf7d61143 100644
--- a/mysql-test/t/tc_heuristic_recover.test
+++ b/mysql-test/t/tc_heuristic_recover.test
@@ -9,7 +9,7 @@
 # All this proves no crashes and effective rollback of the transaction.
 #
 --source include/have_innodb.inc
-# The test logics really requires --log-bin.
+# The test logic really requires --log-bin.
 --source include/have_binlog_format_mixed.inc
 --source include/have_debug_sync.inc
 --source include/not_embedded.inc
@@ -22,14 +22,7 @@ call mtr.add_suppression("Aborting");
 #
 # The "restart" expect-file facility can't be engaged because the server
 #  having conflicting options may not succeed to boot up.
-# Also notice $MYSQLD_CMD is too "static" being unaware of the actual options
-# of the last (before shutdown or kill) server run.
-# That's why $MYSQLD_LAST_CMD that allows for the server new start
-# with more options appended to a stub set which is settled at this very point.
---let $mysqld_stub_cmd= $MYSQLD_LAST_CMD
---let $error_log= $MYSQLTEST_VARDIR/log/mysqld.1.err
---let SEARCH_FILE= $error_log
-set debug_sync='RESET';
+--let SEARCH_FILE=$MYSQLTEST_VARDIR/log/mysqld.1.err
 
 CREATE TABLE t1 (i INT) ENGINE=InnoDB;
 SET GLOBAL innodb_flush_log_at_trx_commit=1;
@@ -55,8 +48,8 @@ SELECT * FROM t1;
 
 # TODO: MDEV-12700 Allow innodb_read_only startup without prior slow shutdown.
 --source include/kill_mysqld.inc
---let $restart_parameters= --innodb-force-recovery=4
---source include/fail_start_mysqld.inc
+--error 1
+--exec $MYSQLD_LAST_CMD --log-bin=master-bin --binlog-format=mixed --core-file --loose-debug-sync-timeout=300 --innodb-force-recovery=4
 
 --let SEARCH_PATTERN= was in the XA prepared state
 --source include/search_pattern_in_file.inc
@@ -65,8 +58,8 @@ SELECT * FROM t1;
 --let SEARCH_PATTERN= \\[ERROR\\] Can\\'t init tc log
 --source include/search_pattern_in_file.inc
 
---let $restart_parameters= --innodb-force-recovery=4 --tc-heuristic-recover=COMMIT
---source include/fail_start_mysqld.inc
+--error 1
+--exec $MYSQLD_LAST_CMD --log-bin=master-bin --binlog-format=mixed --core-file --loose-debug-sync-timeout=300 --innodb-force-recovery=4 --tc-heuristic-recover=COMMIT
 --let SEARCH_PATTERN= was in the XA prepared state
 --source include/search_pattern_in_file.inc
 --let SEARCH_PATTERN= Found 1 prepared transactions!
@@ -76,8 +69,8 @@ SELECT * FROM t1;
 --let SEARCH_PATTERN= Please restart mysqld without --tc-heuristic-recover
 --source include/search_pattern_in_file.inc
 
---let $restart_parameters= --tc-heuristic-recover=ROLLBACK
---source include/fail_start_mysqld.inc
+--error 1
+--exec $MYSQLD_LAST_CMD --log-bin=master-bin --binlog-format=mixed --core-file --loose-debug-sync-timeout=300 --tc-heuristic-recover=ROLLBACK
 
 --let SEARCH_PATTERN= was in the XA prepared state
 --source include/search_pattern_in_file.inc
@@ -88,7 +81,6 @@ SELECT * FROM t1;
 --let SEARCH_PATTERN= Please restart mysqld without --tc-heuristic-recover
 --source include/search_pattern_in_file.inc
 
---let $restart_parameters=
 --source include/start_mysqld.inc
 
 --let SEARCH_PATTERN= was in the XA prepared state
diff --git a/scripts/mysqld_safe.sh b/scripts/mysqld_safe.sh
index 3a06f7e4858..b06106ceae6 100644
--- a/scripts/mysqld_safe.sh
+++ b/scripts/mysqld_safe.sh
@@ -245,7 +245,7 @@ wsrep_recover_position() {
   local euid=$(id -u)
   local ret=0
 
-  local wr_logfile=$(mktemp wsrep_recovery.XXXXXX)
+  local wr_logfile=$(mktemp /tmp/wsrep_recovery.XXXXXX)
 
   # safety checks
   if [ -z $wr_logfile ]; then
@@ -267,7 +267,7 @@ wsrep_recover_position() {
 
   log_notice "WSREP: Running position recovery with $wr_options"
 
-  eval_log_error "$mysqld_cmd --wsrep_recover $wr_options 2> $wr_logfile"
+  eval "$mysqld_cmd --wsrep_recover $wr_options 2> $wr_logfile"
 
   local rp="$(grep 'WSREP: Recovered position:' $wr_logfile)"
   if [ -z "$rp" ]; then
diff --git a/sql/log_event.cc b/sql/log_event.cc
index 177e6c2c775..f57fa7d2718 100644
--- a/sql/log_event.cc
+++ b/sql/log_event.cc
@@ -7670,7 +7670,7 @@ Gtid_log_event::print(FILE *file, PRINT_EVENT_INFO *print_event_info)
   char buf[21];
   char buf2[21];
 
-  if (!print_event_info->short_form & !is_flashback)
+  if (!print_event_info->short_form && !is_flashback)
   {
     print_header(&cache, print_event_info, FALSE);
     longlong10_to_str(seq_no, buf, 10);
diff --git a/storage/connect/CMakeLists.txt b/storage/connect/CMakeLists.txt
index d422b076fc2..dc1f61d0d26 100644
--- a/storage/connect/CMakeLists.txt
+++ b/storage/connect/CMakeLists.txt
@@ -247,7 +247,7 @@ ENDIF(CONNECT_WITH_ODBC)
 #
 # JDBC with MongoDB Java Driver included but disabled
 #
-#OPTION(CONNECT_WITH_MONGO "Compile CONNECT storage engine with MONGO support" ON)
+OPTION(CONNECT_WITH_MONGO "Compile CONNECT storage engine with MONGO support" ON)
 OPTION(CONNECT_WITH_JDBC "Compile CONNECT storage engine with JDBC support" ON)
 
 IF(CONNECT_WITH_JDBC)
diff --git a/storage/innobase/row/row0import.cc b/storage/innobase/row/row0import.cc
index e220c0275bb..1804cba7065 100644
--- a/storage/innobase/row/row0import.cc
+++ b/storage/innobase/row/row0import.cc
@@ -3597,10 +3597,6 @@ row_import_for_mysql(
 
 	if (err != DB_SUCCESS) {
 		return(row_import_error(prebuilt, trx, err));
-	}
-
-	if (err != DB_SUCCESS) {
-		return(row_import_error(prebuilt, trx, err));
 	} else if (cfg.requires_purge(index->name)) {
 
 		/* Purge any delete-marked records that couldn't be