add_sync: fix checking for existing dbgroup; add back in confirm_commit

Make tables=all (e.g. for add_sync) add only tables with pkeys or unique indexes; also excludes the bucardo schema.

Use savepoints when removing databases, in case we need to use the evals

Add new ENV  BUCARDO_CONFIRM which sets bcargs->{confirm} inside of the bucardo program.

Mild doc updates

Use smallint, integer, and bigint rather than int[248] for the column types in the goat table.

Allow add_database to take more than one arg so we can add multiple databases at once via command line

Allow the 'dbs' argument to specify a dbgroup when doing "add_sync", for backwards compatibility.

Remove few instances of trailing whitespace

Trailing whitespace check for Bucardo.pm too

Quick trailing whitespace testing

Make add_sync much smarter and clearer. Should now be able to add dbgroups, dbs, tables, and relgroups more intuitively.

Use -l to check for a symlink.

Not -s. From [-X.pod](http://perldoc.perl.org/functions/-X.html):

    -s  File has nonzero size (returns size in bytes).
    -l  File is a symbolic link.

Bug introduced in 9a1c0dbf.

Allow args to use : or = as delimiters.
Allow more aliases for tables in add sync calls.

Set the log prefix immediately after forking, based on the passed-in type

Spellcheck and linty cleanups

Make sure we import our methods!

Tabs to spaces

Do not like Encode::Locale dependency, but since it is here, might as well use its method for argv decoding.

Change wording in examples as we use target, not slave. Not even sure the commented examples in bucardo are valid anymore!

Tweak some redo logic

We need target column on index to be text_pattern_ops, as we are going to start looking for partials.

Someone fix this test, please.

No more ^T: in the main distinct delta gathering clause: there will be a new way to do things, linked to the new partial sync system.

First swing at handling dead databases in a smooth manner.
If a database is unreachable, we mark it as stalled. The sync containing it is marked as stalled as well.
This should allow other syncs to continue even if databases from another sync are down.
We also re-check stalled databases on startup and restore things to an active state if we can.
Still todo:
* tests
* handle other dbi databases
* handle non-dbi databases
* have mcp periodically check if we can restore stalled dbs/syncs.
* cleanup/refactor the code

Do a check for a stopfile after MCP sleeps. Prevents infinite loop if something goes wrong before the MCP can spin up all the way.

Minor fixes

Add support for "bucardo reopen" command, which will cause all processes to re-open any existing file handles.
Useful for log rotations.

Framework for new 'reopen' command

Print the last git commit information on startup

Don't try to use pg_cancel if things blew up, simply rollback and set async_active off!

Make sure we only update bucardo_truncate_trigger if we really need to, per report of severe bloat on the mailing list.

Prevent uninitialized warnings from no db.service on validate, per mailing list complaint.

Rename the somewhat confusing bucardo version configuration variables.
bucardo_version becomes bucardo_initial_version
bucardo_current_version becomes bucardo_version

Mild TODO cleanup

No sense it sending notification if the db is down

More tweaking to async code

Random comments about new plan

Fix typo

Debugging to solve pg_result errors.

Grabbed incorrect hash - was causing the async_status to not get checked and cleared properly on serialization failures occuring at the pg_result point. Shoudl clear up the recent problem seen on the mailing list.

Sig for 4.99.12

Bump version to 4.99.12

Better tracking of async status

Better service arg parsing

readlink can sometimes retrurn undef, even if -s is true, so make a safety check

Mild tweaks to log_level notifications

Support symlinked bucardo executable when looking for Bucardo.pm and bucardo.schema
Allow use of "all" and "tables=all" when creating a herd/relgroup
Don't keep retrying connection if in batch mode

sqlite: do a BEGIN EXCLUSIVE TRANSACTION in force-lock mode.

Put the force-lock-file in the piddir, rather than the globally writeable /tmp

Take out the session level isolation level for mysql - our per transaction calls should be good enough.

Use READ WRITE isolation levels wherever we can.

Have kid_pid_start notification happen on the main channel, not per database

Gather the maximum transaction time for all tables in a single UNION query,
rather than making separate SQL calls for each table.
Per suggestion on mailing list from Hans van der Riet

Add error NOTIFY when reload fails

Exit right away if the delta test fails, for better debugging.

When debug is on, write out a small text file to help connect to the test databases via psql.

Clean up confusing comment

Cleanup the temporary service files

Move kid_pid_stop to global listening, not db-based.
Do a ping check before trying to cleanup databases on the kid exception handler.

Better debugging for notifys

Rewrite the delete_rows function.
Loop through and make sure we have the maximum number of async targets running at all times.
Moved much of the logic to per-target attribs.

Debug the initdb call during testing

Better tweaking for pgservice items

Remove debugging.

Better munging if we are using a service

Ignore any local .bucardorc files.

Remove useless comment

Bump copyright to 2014

Add test for 'bucardo upgrade'

More bucardo.upgrade fixes

Now bucardo upgrade run against a database that doesn't need any changes
correctly reports that no changes are necessary

If we are using a service file, make sure we don't default slurp in host and port env variables

git should ignore service testing files

Modify syntax so the upgrade process can understand it

Make the dbservice column have an empty default, so we don't have to start checking "defined or length..." everywhere

Revert comment, so it stays in sync with reverted code

Finish supporting non-ASCII table and column names

Bucardo now accepts pg_service for databases

service strings are working; still needs tests

Always use async when doing deletes. The previous way had a thinko - this should ensure large deletes run concurrently on different hosts.

Make install quit ignoring dbname and dbuser options

... again.

Don't assume the bucardo database exists on install

Revert "Make install quit ignoring dbname and dbuser options"

I should probably have run the test suite before committing...

This reverts commit 39dcde241abcdfcc45415d2eb210d8b2c694f479.

Make install quit ignoring dbname and dbuser options

s/vaccuum/vacuum/

Make our statement_chunk_size default match up.

Debugging code

Password should default to undef, not an empty string

Gotta have log levels on all glog() calls.

Version bump to 4.99.11

Eval everything.

Someone cannot count.

Fix typo in comment.

Check @tableoids, not tablelist, as the latter may be empty (inside validate_sync)

Meta things for the new dev/ directory

Various tests for developers

Better makedelta test. Need to move these somewhere else.

Fix up some of the makedelta logic.

Better handling of inactive databases.

Add a fifth inactive database to ensure we handle inactive targets gracefully.