Skip to content
This repository has been archived by the owner on Aug 21, 2023. It is now read-only.

Commit

Permalink
fix getListTableTypeByConf
Browse files Browse the repository at this point in the history
  • Loading branch information
Ehco1996 authored and ti-chi-bot committed Aug 24, 2021
1 parent 8332923 commit ceb9bc1
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 2 additions & 0 deletions tests/run.sh
Original file line number Diff line number Diff line change
Expand Up @@ -54,3 +54,5 @@ else
script="tests/$test_case/run.sh"
run_case_by_fullpath "$script"
fi

echo "Passed integration tests."
2 changes: 1 addition & 1 deletion v4/export/dump.go
Original file line number Diff line number Diff line change
Expand Up @@ -857,7 +857,7 @@ func getListTableTypeByConf(conf *Config) listTableType {
if conf.Consistency == consistencyTypeLock {
// for consistency lock, we need to build the tables to dump as soon as possible
listType = listTableByInfoSchema
} else if conf.Consistency != consistencyTypeNone && matchMysqlBugversion(conf.ServerInfo) {
} else if conf.Consistency == consistencyTypeFlush && matchMysqlBugversion(conf.ServerInfo) {
// For some buggy versions of mysql, we need a workaround to get a list of table names.
listType = listTableByShowFullTables
}
Expand Down

0 comments on commit ceb9bc1

Please sign in to comment.