Try harder to collect and emit errors on validation failures.

There's no need to wait for a NOTIFY, as `validate_sync()` runs synchronously.
So just check the return value for errors, and use excpetion handling to catch
other kinds of errors. This allows us to catch and emit all errors and also
allow execution to finish validating all syncs (should it be validating
multiples). Closes #65.

Improve the auto-creation of the "bucardo" role.

When adding a database, that is. The old code led to confusing errors, where
the user would be told that Bucardo failed to connect as user "postgres" when
you had not, in fact, asked it to. So log more informat as it goes along,
noting failure to connect as "bucardo", trying to connect as "postgres", and
showing success or failure at doing so. This makes things much clearer to the
user what's going on -- and is less fussy, to boot.

Resolves #63.

More bypassing of track_rates

Skip the not-working-now track_rates section.

Merge in patch to ensure objects from different databases but having the same
names are treated distinctly.

Log verbose instead of debug.

For serialization and deadlock failures.

Log loglevel as a string.

Make it clearer which handles to log.

Topic, not lexical.

Log only errors.

Try harder to find initdb when testing.

Add some text fields to the dbgroup and herd tables for future use in "cloning"

Make the inactivate sub a bit smarter. May fix the recurring doublefree/coredump VAC issue.

Fix inaccurate comment.

Changing a sync status does not load or unload it.

So document that fact. We may or may not want to change this in the future.
Ref #55.

Update sync status on activate/deactivate.

Resolves #55.

Document strict_checking for tables, too.

Allow it to be passed to `add table` as well as `update table`.

Ref #57.

Document strict_checking and add it to `add sync`.

Resolves #57.

Add the reload_config_timeout config.

And put it to use in the reload_config command. Also provide more context in
the error message when it does timeout. Closes #67.

Change default reload_config timeout to 30s.

Ref #67.

Purge unknown tables from bucardo_delta_targets.

They seem to get left behind when a table is dropped. Maybe other reasons,
I honestly don't know.

Don't show serialize/deadlock errors unless we are LOG_VERBOSE, per mailing list discussion.

Switch time zone to GMT, as we did in Bucardo4

In `status`, if last run is "empty", call it "good".

Two reasons for this:

* An empty run is not a failure, but "Empty" is kind of distracting
  in the output.
* It is useful, if you just kicked a sync, to see that it ran in the output of
  `status`, even if it was empty.

State can also be "Unknown".

Though I am not sure how it gets in that state...

Note state value for currently-running sync.

End syncs on deadlock and serialization failure.

When I added the serialization failure-handling code, I neglected to notice
that it left orphaned `syncrun` rows with no `ended` value. This confused
later syncruns -- like the one that would run after the 0.5s sleep after a
serialization failure. The code was rolling back the main (Bucardo) database
connection, but the syncrun was already committed elsewhere (so that other
processes could see that a sync is in progress, of course; duh!).

So modify the code to instead call `end_syncrun()` and declare the sync
failed, but, of course, with a status recording the seriazliation failure or
deadlock.

Listen for autokicks on sources for the sync, not all sources.

I had a database in two dbgroups (and syncs), once as a target and once as a
source. On startup, `validate_sync()` was processing the sync where it was a
target, first, and that was the role saved for it in `$self->{sdb}`. Then,
when `validate_sync()` iterated over `$self->{sdb}`, it thought it was a role,
and so did not have the MCP listen for kicks.

Not only that, but sources from other, previously-validated syncs were
unnecessarily listening for kicks!

So change the code to iterate over the list of DBs for the current call to
`validate_sync()`, not all dbs for all syncs, when figuring out whereq to listen
to for kicks.

Note that the use of the same db object in multiple db groups and syncs *may*
cause issues in other places. It would be smart to audit all code that checks
the value in `{role}` for each hash in `$self->{sdb}` and figure out if it is
appropriate. Since the same db can have different roles in differnt syncs, it
might make sense to record all of its roles and to check all of them, rather
than just rely on the first role found.

Revert "Make kick triggers consistently use the non-payload version."

This reverts commit 25973f4b5ba797568467632b79e8129ffe007f49. I'm pretty sure
this was not the problem, since I have syncs that *do* work with this
approach.

The problem I'm having with a NOTIFY being ignored is that the database in
question is used in two db groups, once as a source, another as a target. As
it validates, it finds the target record before the source, and stores that
one away. Then the autokick is ignored, because Bucardo thinks the DB is a
target, not a source. Working on a fix for that now.

Check for empty state, too.

Remove listen_payload in unlisten_all().

Add daemon role to application name.

Make kick triggers consistently use the non-payload version.
May want to have MCP be able to recognize both versions for backwards compat.

Fix `reload config` timeout.

First, allow `reload config 30` to work. Resolves #66.

Second, actually *capture* the timeout and wait that long, instead of just
five seconds every time.

Remove comment line about spacing - no longer applies.

Make sure "bucardo help" doesn't try to connect to a database. Per github issue #64

Bump the version to 5.0.0.

Switch default isolation level back to repeatable read

Match process name case-insensitive

Proper debug for isolation levels

Fixes for upgrade magic

Let's do github canonically

Move some things from bugzilla to here

Super minor tweak

VAC handle pg_listener, per list discussion

This should have gone in with the last commit, as well

Improve output, and update tests accordingly

Add a bit more output to identify a database

Add tests and code for update table db filtering

Clean up lots, and get list table working better

Clean up tests

Merge branch 'master' of ssh://bucardo.org/var/lib/git/bucardo into database-specific-names

Fix customcols; all preexisting tests pass

even more tests work now

Fix herd adding logic, with caveats noted in test source

Consolidate methods for finding goats

Allow for auto reconnection of databases for crash testing.

Typo

Merge from master

02-bctl-herd.t passes

The regex looking at ps output when killing children didn't match the children

Avoid warning if no $HOME.

Incremental progress

Refactored %GOAT, and several functions that refer to it
Added beginnings of a new test file

This doesn't work yet, but needs to live somewhere other than just my computer.

Show correct error

Add more explicit support for adding and removing sequences

Attempts to make VAC daemon quieter, per IRC discussions.

Make sure regex matches so "Not needed" doesn't raise a warning.

Allow transaction isolation level to be changed

Better debugging sub

Signature for version 4.99.7

Sometimes the bucardo_delta table can get duplicate entries. Make sure the purge function is not tripped up by that.

Minor Perl::Critic inspired tweaks

Fix comment

Show filename for subroutines that do not pass.

Testing tweaks.

Make sure logdest gets forced to an arrayref.

Tabs to space

Skip generated files from some tests.

Bump version to 4.99.7

Work around for Postgres bug in which notices are not picked up if unlisten * and listens are put in the same transaction!

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

Mild cleanups. Allow foo:bar in addition to foo=bar in the RC files

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.

Don't run FK info query if no FKs

Bump to 2013

Typo

Merge remote-tracking branch 'burbon/fix-update-dbgroup'

Merge remote-tracking branch 'burbon/fix-add-all-tables'

FIX: bucardo update dbgroup do not affect db data

remove unnecessary variable.

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.

FIX: add all tables with table/schema limit
add no_autoabbrev to save '-n' (schema|n=s@ in second GetOptions) from binding to 'no-bucardorc' in first GetOptions

FIX: add all tables with exclude table/schema

Idea from mailing list: combine kick triggers and funcs

Show status, not details. PID is not in the syncrun table anymore.

Better output for "whydie" section.

More incremental work for solving the FK conflict problem.

Track which tables are referencing each other via foreign keys. This is needed for conflict resolution work.

Store the dbgroup rather than the sync inside of bucardo_delta_targets. This makes more sense, as not only are the track_* tables storing the db name, but the main getdelta queries use that as well. Thus, we don't want to store syncs as dbgroups are more unique.

Walked through to make sure all execute() calls are followed by a finish() or a fetchall()