From 3e1e9b842545f698741ed20f45a8e620dd5fd707 Mon Sep 17 00:00:00 2001
From: Sveta Smirnova <sveta.smirnova@percona.com>
Date: Thu, 9 Nov 2023 17:16:43 +0300
Subject: [PATCH]  PT-1860 make pt-osc respect case insesitive lookup on
 Windows and osx #516

- Run update-modules
- Adjusted fix, so it does not create extra variable to hold $EVAL_ERROR
- Adjusted test case, so it runs on Linux
---
 bin/pt-online-schema-change                   | 22 +++++++++++--
 lib/TableParser.pm                            |  6 ++--
 t/pt-online-schema-change/case-insensitive.t  | 31 ++++++++-----------
 .../samples/lower_case_table_names_1.cnf      |  2 ++
 4 files changed, 38 insertions(+), 23 deletions(-)
 create mode 100644 t/pt-online-schema-change/samples/lower_case_table_names_1.cnf

diff --git a/bin/pt-online-schema-change b/bin/pt-online-schema-change
index 6ab96703..9eb6b769 100755
--- a/bin/pt-online-schema-change
+++ b/bin/pt-online-schema-change
@@ -3470,11 +3470,29 @@ sub check_table {
    }
    my ($dbh, $db, $tbl) = @args{@required_args};
    my $q      = $self->{Quoter} || 'Quoter';
+   $self->{check_table_error} = undef;
+
+
+   my $lctn_sql = 'SELECT @@lower_case_table_names';
+   PTDEBUG && _d($lctn_sql);
+
+   my $lower_case_table_names;
+   eval { ($lower_case_table_names) = $dbh->selectrow_array($lctn_sql); };
+   if ( $EVAL_ERROR ) {
+      PTDEBUG && _d($EVAL_ERROR);
+      $self->{check_table_error} = $EVAL_ERROR;
+      return 0;
+   }
+
+   PTDEBUG && _d("lower_case_table_names=$lower_case_table_names");
+   if ($lower_case_table_names > 0) {
+       PTDEBUG && _d("MySQL uses case-insensitive lookup, converting '$tbl' to lowercase");
+       $tbl = lc $tbl;
+   }
+
    my $db_tbl = $q->quote($db, $tbl);
    PTDEBUG && _d('Checking', $db_tbl);
 
-   $self->{check_table_error} = undef;
-
    my $sql = "SHOW TABLES FROM " . $q->quote($db)
            . ' LIKE ' . $q->literal_like($tbl);
    PTDEBUG && _d($sql);
diff --git a/lib/TableParser.pm b/lib/TableParser.pm
index 58b30a66..896a13f4 100644
--- a/lib/TableParser.pm
+++ b/lib/TableParser.pm
@@ -339,9 +339,9 @@ sub check_table {
 
    my $lower_case_table_names;
    eval { ($lower_case_table_names) = $dbh->selectrow_array($lctn_sql); };
-   if ( my $e = $EVAL_ERROR ) {
-      PTDEBUG && _d($e);
-      $self->{check_table_error} = $e;
+   if ( $EVAL_ERROR ) {
+      PTDEBUG && _d($EVAL_ERROR);
+      $self->{check_table_error} = $EVAL_ERROR;
       return 0;
    }
 
diff --git a/t/pt-online-schema-change/case-insensitive.t b/t/pt-online-schema-change/case-insensitive.t
index 3b47bba1..9aa026e1 100644
--- a/t/pt-online-schema-change/case-insensitive.t
+++ b/t/pt-online-schema-change/case-insensitive.t
@@ -12,19 +12,11 @@ use English qw(-no_match_vars);
 use Test::More;
 use Time::HiRes qw(sleep);
 
-$ENV{PTTEST_FAKE_TS} = 1;
-$ENV{PERCONA_TOOLKIT_TEST_USE_DSN_NAMES} = 1;
-
 use PerconaTest;
 use Sandbox;
 require "$trunk/bin/pt-online-schema-change";
 require VersionParser;
 
-use Data::Dumper;
-$Data::Dumper::Indent    = 1;
-$Data::Dumper::Sortkeys  = 1;
-$Data::Dumper::Quotekeys = 0;
-
 my $dp         = new DSNParser(opts=>$dsn_opts);
 my $sb         = new Sandbox(basedir => '/tmp', DSNParser => $dp);
 my $master_dbh = $sb->get_dbh_for('master');
@@ -33,23 +25,25 @@ if ( !$master_dbh ) {
    plan skip_all => 'Cannot connect to sandbox master';
 }
 
-my $q      = new Quoter();
-my $tp     = new TableParser(Quoter => $q);
 my @args   = qw(--set-vars innodb_lock_wait_timeout=3);
 my $output = "";
 my $dsn    = "h=127.1,P=12345,u=msandbox,p=msandbox";
 my $exit   = 0;
 my $sample = "t/pt-online-schema-change/samples";
-my $rows;
 
-# #############################################################################
-# Tool shouldn't run without --execute (bug 933232).
-# #############################################################################
+my $lower_case_table_names = $master_dbh->selectrow_array('SELECT @@lower_case_table_names');
 
-my ($lower_case_table_names) = $master_dbh->selectrow_array('SELECT @@lower_case_table_names');
-plan skip_all => 'MySQL uses case sensitive lookup' if $lower_case_table_names == 0;
+if ( $lower_case_table_names == 0) {
+   # Preparing test setup on Linux
+   diag(`$trunk/sandbox/stop-sandbox 12348 >/dev/null`);
+   diag(`EXTRA_DEFAULTS_FILE="$trunk/t/pt-online-schema-change/samples/lower_case_table_names_1.cnf" $trunk/sandbox/start-sandbox master 12348 >/dev/null`);
 
-$sb->load_file('master', "$sample/basic_no_fks_innodb.sql");
+   $master_dbh = $sb->get_dbh_for('master1');
+   $dsn    = "h=127.1,P=12348,u=msandbox,p=msandbox";
+   $sb->load_file('master1', "$sample/basic_no_fks_innodb.sql");
+} else {
+   $sb->load_file('master', "$sample/basic_no_fks_innodb.sql");
+}
 
 ($output, $exit) = full_output(
    sub { pt_online_schema_change::main(@args, "$dsn,D=pt_osc,t=T",
@@ -78,7 +72,8 @@ is(
 # #############################################################################
 # Done.
 # #############################################################################
-
+diag(`$trunk/sandbox/stop-sandbox 12348 >/dev/null`);
+$master_dbh = $sb->get_dbh_for('master');
 $sb->wipe_clean($master_dbh);
 ok($sb->ok(), "Sandbox servers") or BAIL_OUT(__FILE__ . " broke the sandbox");
 done_testing;
diff --git a/t/pt-online-schema-change/samples/lower_case_table_names_1.cnf b/t/pt-online-schema-change/samples/lower_case_table_names_1.cnf
new file mode 100644
index 00000000..36313cc7
--- /dev/null
+++ b/t/pt-online-schema-change/samples/lower_case_table_names_1.cnf
@@ -0,0 +1,2 @@
+[mysqld]
+lower_case_table_names=1