Skip to content

Commit

Permalink
mysqlctl: remove lower_case_table_names setting
Browse files Browse the repository at this point in the history
Issue #3158
The current value of 0 is definitely incorrect because vitess
treats table names as case-sensitive. It's better we don't set
this variable at all. Let the users decide what it should be.
  • Loading branch information
sougou committed Nov 14, 2017
1 parent 6c87743 commit 3a7ef0e
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 8 deletions.
3 changes: 0 additions & 3 deletions config/mycnf/default-fast.cnf
Expand Up @@ -58,6 +58,3 @@ thread_cache_size = 200
tmpdir = {{.TmpDir}}
tmp_table_size = 32M
transaction-isolation = REPEATABLE-READ
# READ-COMMITTED would be better, but mysql 5.1 disables this with statement based replication
# READ-UNCOMMITTED might be better
lower_case_table_names = 1
3 changes: 0 additions & 3 deletions config/mycnf/default.cnf
Expand Up @@ -56,9 +56,6 @@ thread_cache_size = 200
tmpdir = {{.TmpDir}}
tmp_table_size = 32M
transaction-isolation = REPEATABLE-READ
# READ-COMMITTED would be better, but mysql 5.1 disables this with statement based replication
# READ-UNCOMMITTED might be better
lower_case_table_names = 1

# Semi-sync replication is required for automated unplanned failover
# (when the master goes away). Here we just load the plugin so it's
Expand Down
4 changes: 2 additions & 2 deletions test/keyspace_test.py
Expand Up @@ -22,8 +22,8 @@
import tablet
import utils

SHARDED_KEYSPACE = 'TEST_KEYSPACE_SHARDED'
UNSHARDED_KEYSPACE = 'TEST_KEYSPACE_UNSHARDED'
SHARDED_KEYSPACE = 'test_keyspace_sharded'
UNSHARDED_KEYSPACE = 'test_keyspace_unsharded'

# shards for SHARDED_KEYSPACE
# range '' - 80
Expand Down

0 comments on commit 3a7ef0e

Please sign in to comment.