Merge lp:~akopytov/percona-xtrabackup/bug1183500-2.1 into lp:percona-xtrabackup/2.1

Proposed by Alexey Kopytov
Status: Merged
Approved by: Sergei Glushchenko
Approved revision: no longer in the source branch.
Merged at revision: 619
Proposed branch: lp:~akopytov/percona-xtrabackup/bug1183500-2.1
Merge into: lp:percona-xtrabackup/2.1
Diff against target: 148 lines (+58/-25)
4 files modified
test/inc/common.sh (+48/-15)
test/t/bug1130627.sh (+4/-4)
test/t/bug766607.sh (+5/-4)
test/t/tar4ibd_symlink.sh (+1/-2)
To merge this branch: bzr merge lp:~akopytov/percona-xtrabackup/bug1183500-2.1
Reviewer Review Type Date Requested Status
Sergei Glushchenko (community) g2 Approve
Review via email: mp+172190@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Alexey Kopytov (akopytov) wrote :
Revision history for this message
Sergei Glushchenko (sergei.glushchenko) wrote :

Approve

review: Approve (g2)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'test/inc/common.sh'
--- test/inc/common.sh 2013-05-07 13:32:47 +0000
+++ test/inc/common.sh 2013-06-29 03:20:36 +0000
@@ -351,25 +351,58 @@
351}351}
352352
353########################################################################353########################################################################
354# Shutdown cleanly server specified with the first argument
355########################################################################
356function shutdown_server_with_id()
357{
358 local id=$1
359 switch_server $id
360
361 vlog "Shutting down server with id=$id..."
362
363 if [ -f "${MYSQLD_PIDFILE}" ]
364 then
365 ${MYSQLADMIN} ${MYSQL_ARGS} shutdown
366 else
367 vlog "Server PID file '${MYSQLD_PIDFILE}' doesn't exist!"
368 fi
369
370 # unlock the port number
371 free_reserved_port $MYSQLD_PORT
372
373 reset_server_variables $id
374}
375
376########################################################################
354# Shutdown server with id=1 cleanly377# Shutdown server with id=1 cleanly
355########################################################################378########################################################################
356function shutdown_server()379function shutdown_server()
357{380{
358 switch_server 1381 shutdown_server_with_id 1
359382}
360 vlog "Shutting down server with id=1..."383
361384########################################################################
362 if [ -f "${MYSQLD_PIDFILE}" ]385# Force a checkpoint for a server specified with the first argument
363 then386########################################################################
364 ${MYSQLADMIN} ${MYSQL_ARGS} shutdown387function force_checkpoint_with_server_id()
365 else388{
366 vlog "Server PID file '${MYSQLD_PIDFILE}' doesn't exist!"389 local id=$1
367 fi390 shift
368391
369 # unlock the port number392 switch_server $id
370 free_reserved_port $MYSQLD_PORT393
371394 vlog "Forcing a checkpoint for server #$id"
372 reset_server_variables 1395
396 shutdown_server_with_id $id
397 start_server_with_id $id $*
398}
399
400########################################################################
401# Force a checkpoint for server id=1
402########################################################################
403function force_checkpoint()
404{
405 force_checkpoint_with_server_id 1 $*
373}406}
374407
375########################################################################408########################################################################
376409
=== modified file 'test/t/bug1130627.sh'
--- test/t/bug1130627.sh 2013-03-07 11:48:43 +0000
+++ test/t/bug1130627.sh 2013-06-29 03:20:36 +0000
@@ -5,7 +5,9 @@
5. inc/common.sh5. inc/common.sh
6. inc/ib_part.sh6. inc/ib_part.sh
77
8start_server --innodb_file_per_table8MYSQLD_EXTRA_MY_CNF_OPTS="innodb-file-per-table"
9
10start_server
911
10require_partitioning12require_partitioning
1113
@@ -27,9 +29,7 @@
27PARTITIONS 10 */;29PARTITIONS 10 */;
28EOF30EOF
2931
30# Force a checkpoint32force_checkpoint
31stop_server
32start_server --innodb_file_per_table
3333
34# Test that specifying partitions with --include works34# Test that specifying partitions with --include works
35innobackupex --no-timestamp --include='^test.*#P#p5' $topdir/backup35innobackupex --no-timestamp --include='^test.*#P#p5' $topdir/backup
3636
=== modified file 'test/t/bug766607.sh'
--- test/t/bug766607.sh 2013-03-07 11:38:14 +0000
+++ test/t/bug766607.sh 2013-06-29 03:20:36 +0000
@@ -1,6 +1,8 @@
1. inc/common.sh1. inc/common.sh
22
3start_server --innodb_file_per_table3MYSQLD_EXTRA_MY_CNF_OPTS="innodb-file-per-table"
4
5start_server
4load_dbase_schema incremental_sample6load_dbase_schema incremental_sample
57
6# Backup dir8# Backup dir
@@ -15,8 +17,7 @@
15FLUSH LOGS;17FLUSH LOGS;
16EOF18EOF
1719
18stop_server20force_checkpoint
19start_server --innodb_file_per_table
2021
21vlog "Making incremental backup"22vlog "Making incremental backup"
22innobackupex --incremental --no-timestamp --incremental-basedir=$topdir/backup/full $topdir/backup/delta23innobackupex --incremental --no-timestamp --incremental-basedir=$topdir/backup/full $topdir/backup/delta
@@ -42,6 +43,6 @@
42cd $topdir43cd $topdir
4344
44vlog "Data restored"45vlog "Data restored"
45start_server --innodb_file_per_table46start_server
4647
47run_cmd $MYSQL $MYSQL_ARGS -e "SELECT * FROM t" test48run_cmd $MYSQL $MYSQL_ARGS -e "SELECT * FROM t" test
4849
=== modified file 'test/t/tar4ibd_symlink.sh'
--- test/t/tar4ibd_symlink.sh 2012-06-05 12:35:33 +0000
+++ test/t/tar4ibd_symlink.sh 2013-06-29 03:20:36 +0000
@@ -10,8 +10,7 @@
10load_dbase_data sakila10load_dbase_data sakila
1111
12# Force a checkpoint12# Force a checkpoint
13stop_server13force_checkpoint
14start_server
1514
16# Copy some .ibd files to a temporary location and replace them with symlinks15# Copy some .ibd files to a temporary location and replace them with symlinks
1716

Subscribers

People subscribed via source and target branches

to all changes: