Update the RPM support files.

Includes a patch that puts all the log files into /var/log/bucardo by default.

Move distribution-supporting scripts to `dist`.

Fix test failure introduced in f77fdbfd88b8.

BucardoTesting now throws an exception on `qx()` failure, and prepends its own
info to the returned message.

Remove redundant LISTEN.

The value of `"bucardo_syncdone_$sync"` is exactly the same as the `"$done"`
two lines before. And `$s` was unused.

Test all return values of `ctl()` in postgres tests.

I am still getting test failures, but they are less noisy now that I have it
die after a `sync` fails. The current reason for the failures is that the call
to `goto KID` added in ea4923155f comes before the definition of the KID
label.

In passing, make it clearer that BucardoTesting forces a timeout even when you
pass a `0` to `ctl()`.

Work around missing database handles.

This was causing tests to fail, because the KID was choking on something, and
so cleaning up, closing all database handles, but some were simply not there.
So add code to notice this condition and log a warning.

Unknown is why this happens in the first place. Bug?

Update test count.

Don't activate msync.

It makes the tests BAIL OUT.

Fix exit value of `update` command.

It was calling `exit 1` on sucess, because it expected the various `update_*`
functions to return booleans. They don't. It looks like they return undef on
success and `exit` themselves on failure.

So change the code to only exit with failure if the thing we're asked
to update is unknown. Ottherwise, call the proper `update_*` function
and `exit 0`.

Discovered by changing the `ctl` test method die if `qx()` exits abnormally
and starting to check the return value of `ctl` more often.

Fix invalid sync activation.

It was trying to start a sync named "mtest", and there is no such sync.
So test the return value of `ctl()` there and fix it.

This, BTW, causes the tests to completely bail out. Maybe msync should
not be activated?

Quick implementation of the retry on serialization plan laid on on the mailing list.

It is possible (although very dumb) to set kid_serial_sleep to 0, so base our decision on definededness not == 0.

When scanning for errors, no need to keep scanning once we find an interesting one (deadlock or serialization) because there can be only one such.

Sleep less; report errors.

I sometimes deleted the test data directories without shutting down pg. I
would then get failures about not being able to find the postgresql.conf file.
Why? Because I was out of shared memory, so the initdb failed. But there were
no initdb errrors. So the `die $res if $? != 0;` line shows those failures.

Meanwhile, working on trying to figure out serialization failures, it seems
silly for it to sleep 10s while testing, so have it sleep for just 1s.

Remove arg specifier.

Silence warning about active statement handles.

By calling `finish()` on them.

Don't die when Postgres is down.

Noticed during tests that sometimes when the `t/20-postgres.t` tests failed,
the `bucardo.mcp.pid` file was left in place. Even though the MCP was gone.
This prevented subsequent tests from running, as they couldn't start the MCP
(Bucardo thought it was still running, because the PID file was still there).

Looking through the log, I noticed that the connection to the database was
dying in `cleanup_mcp()`. Not sure why, but the log said:

    (78512) [Mon Oct 22 12:01:17.638 2012] #0578 MCP Warning: Killed (line 6722): DBD::Pg::db selectcol_arrayref failed: server closed the connection unexpectedly
     This probably means the server terminated abnormally
     before or while processing the request. at /Users/david/dev/postgresql/bucardo/blib/lib/Bucardo.pm line 6722.

That was dying, and the subsequent lines that actually delete the PID file
were therefore never run.

This commit wraps the call to the database in an `eval{}`. If it dies, the
database shutdown time is set to "unknown". We skip further database
interactions when the time is unknown. This allows the shutdown to continue
without exiting, and the MCP PID file gets cleaned up.

Resolves #26.

Don't start bucardo when no syncs while testing.

This seems to prevent an error where Bucardo won't start, even though it
waited. I suspect this is becaust earlier tests run that start Bucardo with no
syncs and don't bother to stop it again, because they expect it to exit, as it
did prior to cc92a976e00df6a5182745b47793b44401e6b336.

Also, don't shut down Postgres on test exit. It just seems to confuse things.

Shut down on END instead of DESTROY.

Take the hint and shutdown fast.

Fast mode does not wait for clients to disconnect, which is what we want.

Try hader to shut down Postgres.

Use `pg_ctl stop`, as it seems to be more reliable: I kept seeing postgres
processes running after the tests exited. Also, add a DESTROY to be sure to
shut them all down before exiting.

Update `show_why_sync_killed()`.

At least I think so. It was still querying `bucardo.q`, which was removed in
 0763c281. I replaced its calls with the columns that look more or less
 parallel, although `pid` is gone. A review would be appreciated.

I got errors about it not working while running failing tests. Still trying to
chase down the underlying failures...

Revert moving the start file to the debug dir.

Was moved in 56ea2b2b4b0ec63383a32e8c886439a451b88929. But this turned out to
confuse things, such that tests were unable to shut down Bucardo, because the
server and the client had a different idea where the file belongs. This is
most likely the reason I was seeing stop failures as reported in GitHub
Issue #11.

Now that I understand that its location can be set via `bucardo set
reason_file`, I am just documenting that.

Ignore bucardo.restart.reason.*.

Priority currently used only for display.

Document `checktime`.

Document `overdue`.

Mention GitHub issues, link to Mail list.

Remove `limitdbs`.

Was only in 5.0 betas, but I added drop column support to `upgrade()`, anyway.
Works nicely.

Document aliases.

Finish basic documentation of all commands.

Some more commands implemented:

* restart
* message
* reload
* update

Plus fixed the synopses for activate and deactivate, and listed the possible values for
the standard_conflict parameter.

Done editing existing commands.

Document `remove` and update `reload config`.

Document adding `customname`, `customcols`, and `customcode`.

Had to do quite a bit of Googling and code archaeology to figure these out.
Hope they're correct. Corrections appreciated, especially for `customcode`,
which seems to be woefully underdocumented up to now.

Document `add herd` and `add sync`.

Quite a few questions about some of the parameters to `add sync`, demarcated by `XXX`s.

While at it, add a brief intro discussing options and parameters.

Update docs on adding tables and seuences.

Start filling out gaps in the docs.

And correcting apparent inaccuracies and modernizing the Pod a bit. This
commit Updates docs through `add db`. Notable changes:

* Remove the bit in the `start` docs about how it restarts Bucardo. The code
  does not appear to support that, and the `restart` command seems to have it
  covered.
* Remove the "--name" bit from the `start` and `stop` commands. There is no
  `--name` option, so I just found it confusing.
* Itemize the types supported by `list` in a Pod list.
* Itemize all the objects supported by `add`.
* Itemize all of the options to `add db`.

Proof-reading for inaccuracie would be greatly appreciated.

Stub missing commands in docs.

The verb is now split from the noun at line 219.

Change default reason_file name to end in .txt.

Because it is not a log file, only the most reason reason is written
to it.

But also modify the rason log file to replace the extension from the reason
file with `.log`. So it should no longer be named
`bucardo.restart.reason.log.log` by default.

In other words, the names are:

Reason File: `bucardo.restart.reason.txt`
Rason Log:   `bucardo.restart.reason.log`.

Closes GitHub issue #9.

Move the reason file to the debug directory.

Which is `.` by default, so in the default case, it will be the same as the
current behavior. Ref: GitHub Issue #9.

Skip the .init and .spec files.

They are not Perl. Maybe distribution-related stuff should go into another directory? `dist` perhaps?

Add --exit-on-nosync option.

And change the default so that, if there are no active syncs, by default
bucardo will *not* exit. Pass this option to make it exit anyway. Per
discussion; Closes #8

Move all but two blocker to-dos to GitHub issues.

They are still here, but now with links to GitHub. Could not find the post reporting the bug that checks the wrong tables, so I did not add that one. The other one I did not add was answering all mail list messages.

Add a Linux init script and RPM spec file.

The latter is forked from Devrim's pgrpms spec file.

Add chkconfig support to the start script.

Clean up TODO, add some links.

Workarounds for empty primary key join/split problem found by Bill on mailing list.

Remove debug detritus.

Change PGIN to IN; consolidate flat and flatpg modes for now.

and another.

oops.  missed a spot.

consolidating IN and PGIN blocks in delete_rows

Prevent empty trailing items from being ignored when doing a split, especially for the list of primary keys. This should fix a problem reported on the mailing list giving the error "arguments of row IN must all be row expressions"

Quick TODO additions

Since we use MAX(), and the table may have no rows, we need to handle getting back a null, which means that $epoch may be undefined.

Fix some syntax and other problems; simple tests now show 8.1 works as a valid target database.

Force time zone to uppercase GMT for old servers.

Check for cases in which the sync column of bucardo_delta_targets does not exist when doing validate_sync.

Add a single apostrophe to text values for additional quoting checks.

Report proper test failure context.

Added a few of these lines to BucardoTesting.pm:

    local $Test::Builder::Level = $Test::Builder::Level + 1;

Because otherwise, Test::Builder reports the failure from the function or
method call in BucardoTesting.pm, rather than the calling `*.t` file. This
will make it easier to find the file and line of the actual test failure.

Add databas now works with the same parameters.

Greg says, via IRC, "We now allow that, for cool in-place table trimming." So
it seems as though the tests for it should be updated. And so they now are.

The pg_stat_activity PID column name has changed on 9.2.

So detect if we're on 9.2 or higher, and use the new name, `pid`. Still use
the old name, `procpid`, for older versions of PostgreSQL.

Turn list of prereqs into a Pod list and add boolean.

Report reason for compile failure.

Tests for same-database

Allow databases to have the same dsn, as we are moving towards in-database replication.

Testing tweaks

Note recent bug

New siganture for 4.99.5

Update MANIFEST

Bump to version 4.99.5: get ready for next beta

Minor sprintf fix to bucardo

We have column names with underscores, thank you very much.

Make the summary usage message more consistent with the man page.
Items are listed in similar order on both pages. Additionally,
customname, customcols and customcode are mentioned in the usage
message. Finally the usage information for customname, customcols
and customcode are added to the man page.

Testing tweaks.

Switch back to Redis, as Redis::Client does not die smoothly on connection errors.

Make sure we remove the tschema test schema in our tests.

Allow spaces in customnames

Tentative support for multi-column primary keys in MongoDB

Changes file cleanups

Do not map NULL values to Mongo but simply drop them, as MongoDB is schemaless.

Quick inline notes on how to handle common data mismatch problem.

Chunk up the MongoDB deletes so as not to overwhelm the server on large tables.
May still require changing statement_chunk_size

Should be setting, not value

Better lookup for old bucardo_sequence tables.
Per mailing list question from Ed Snajder

Credit last change.

Add 'bucardo list all' to list everything at once.

Add "one logfile per sync" idea from the ML.

Fix up table names now that we are capping test3

Try again for a unique var name

Don't use variable names when a column name already exists - some versions of plpgsql will get confused.

Show full name of relation when giving deadlock details. Fix missing comma.

Spellcheck tweaks

Finish up the Redis work, including a new test.
Also switch from the Redis Perl driver the Redis::Client, as the latter has hash support.

Minor typos.

Minor TODO updates.

Add support for MariaDB

Testing tweaks for SQLite