use $sth->rows() to get number of rows.

From the DBI manpage:

'A DBI statement handle execute always returns true regardless of the number of rows affected, even it's zero." It only returns the number of rows affected for non-"SELECT" statements.'

This patch ensures that we get the proper count by calling $sth->rows() instead of relying on $sth->execute.
remove unnecessary variable.
FIX: bucardo update dbgroup do not affect db data
Merge remote-tracking branch 'burbon/fix-add-all-tables'
Merge remote-tracking branch 'burbon/fix-update-dbgroup'
Typo
Bump to 2013
Don't run FK info query if no FKs
Make sure that when verifying columns, etc. across source databases, we only check source databases used in the current sync. Fixes some complaints from the mailing list.
Mild cleanups. Allow foo:bar in addition to foo=bar in the RC files
Make sure logdest works if specified in an rc file.
Have reload_config check that Bucardo is up and running first
Allow wait_for_notice to have a timeout
Work around for Postgres bug in which notices are not picked up if unlisten * and listens are put in the same transaction!
Bump version to 4.99.7
Skip generated files from some tests.
Tabs to space
Make sure logdest gets forced to an arrayref.
Testing tweaks.
Show filename for subroutines that do not pass.
Fix comment
Minor Perl::Critic inspired tweaks