Merge branch 'bug306'

Conflicts:
doc/adminguide/slonyupgrade.sgml

release notes update

bug 301 - adding a missing PQclear to fix a memory leak

This PQclear was added by Jan to the patch he posted in bugzilla.
Free the result set when dealing with the error.

Merge branch 'bug301'

Merge branch 'bug299'

Conflicts:
doc/adminguide/slonyupgrade.sgml

add in a check/test to ensure that the backup node in a failover is a forwarding subscriber

If the backup node is not a forwarding subscriber to all of the sets from the origin
then the failover can't complete.  It is best to check this early on and warn the user
before the failover actually starts.

add documentation that mentions that the filenames of installable files have changed

The table definitions we create on a 2.1 -> 2.2 upgrade should match the updated DDL in slony1_base.sql
This is to fix bug 306

add in a check/test to ensure that the backup node in a failover is a forwarding subscriber

If the backup node is not a forwarding subscriber to all of the sets from the origin
then the failover can't complete.  It is best to check this early on and warn the user
before the failover actually starts.

additional dstring_free() to fix an additional memory leak in sync_helper

Lots of fixes to anachronisms in FAQ

Fix typo

More textual changes

More fixes to anachonisms

add documentation that mentions that the filenames of installable files have changed

Shift some anachronisms to the Obsolete section

Tweak to release notes

documentation sgml fixes

updating minor upgrade instructions to reflect that slony server files are now versioned
and both versions can co-exist in the $libdir and $sharedir at the same time

documentation changes to reflect incompatibilities between slony 2.1.x and 2.2.0

array_length() doesn't exist on PG 8.3 use array_upper() instead

memory leak fixes in sync_helper

dstring_terminate does not call free() to deallocate memory dstring_free() does

documentation sgml fixes

updating minor upgrade instructions to reflect that slony server files are now versioned
and both versions can co-exist in the $libdir and $sharedir at the same time

documentation changes to reflect incompatibilities between slony 2.1.x and 2.2.0

don't process a set if sl_setsync has the initial :1:1: snapshot id
this snapshot id is reserved for setting up the value indicates that
an even (ie the ACCEPT_SET) is still pending.

Also be explicit about the origin this remote worker is querying
sl_setsync for.

bug 299 - put the WAKEUP event at the front of the event queue instead of the back.

This will prevent the remoteWorker thread for an old-origin from pulling
the set that was just moved if a SYNC event is already queued

updating release notes, bug 296,297,298

Merge branch 'bug298'

Merge branch 'bug297'

Small fixes for failover code.

removing stray quote

more documentation items

add DDL_SCRIPT translation to the logshipping_toinsert

Initial version of a perl script to convert the new
COPY protocol style log shipping files to pure
SQL ones (INSERT,UPDATE,DELETE,TRUNCATE..)

documentation updates for log shipping

Make the slony_logshipper daemon have a proper page in the reference section
like slon and slonik.  Clean up the rest of the logshipping instructions to be
more of a set of instructions versus question/answer format.

bug 297 - fix test_slony_state-dbi.pl so that it works with PG 9.2

Also fix the warning emails for sl_confirm and sl_event checks so that they
actually treat problem conditions as problems.

Add a test to LogShipping.js that tests DDL on logshipping nodes

bug 298 - process DDL_SCRIPT events from the log shipper SQL files

improve the error case detection of slony_logshipper

This commit will give a better error message when the log shipper connects to
a databse without the sl_archive_tracking tables.

Documentation fixes for EXECUTE SCRIPT

ensure that the transactions are closed as part of the FAILOVER before doing a MOVE SET

There were some code paths that could leave a transaction open such that when we later
got to a second set or node the LOCK SET could fail.

Also remove an unused if condition

Save the return code for the next iteration

If the moveSet fails as part of a failover we continue processing
for the other nodes.  This fix make sure that we remember any failures
and slonik will return a non-zero return code when finished

fixing unit test, issue a sync to make sure the nodes have
caught up before doing the compareDb

documentation updates (FAILOVER)

removing unused variables introduced in the previous commits

DROP NODE will now pass all nodes being dropped to the stored functions and event

This allows slony to detect if a provider that is being dropped
only provides the subscription to some other node that is also
being dropped.

Also update the unit tests to reflect the new method
of FAILOVER involving provider-only nodes

bug 296 - part 1 FAILOVER fixes

This change allows a non-origin node to be specified on the FAILOVER command
so that subscriptions for these nodes are not redirected to the backup node.

An array of all the failed nodes (in addition to the one being failed over at this
stage of the process) is now passed to many of the failover stored functions

This commit also adds the SiteFailover unit test that tests multiple nodes in a site
failing over when the cluster does not have a complete path network

adding documentation note about 8.3 having different table names for pgbench

removing documentation note that applies to PG version prior to 8.0
This version of slony requires at least 8.3

more consistent documentation formatting

bug 294 - fixing documentation error - we just added set 2

Merge branch 'master' of ssh://git.postgresql.org/slony1-engine

Fix for bug291.

The apply trigger messed up the query if a row
was updated to all the same values in case a
table has a multi-column primary key.

Add a test case to trigger bug 291.

Varlena material has shifted to new location

only include malloc.h on WIN32 - FREEBSD (and others?) have replaced malloc.h with stdlib.h

updating the name of the version for the functions in the win32 export file

add PG VERSION based switch for HAS_LOOKUPEXPLICITNAMESPACE_2  to the MSVC config file

include malloc.h - alloca() is defined here on win32

updating versions for 2.2.0 beta 4

Merge branch 'execute-script-fixes'

Conflicts:
RELEASE

release notes for bug 252

Fix typo in '<event pending>' conninfo string.

Change the temporary conninfo on clone prepare to '<event pending>'
for consistency.

release notes update

fixing merge error - string replacements in SQL

Merge branch 'master' into execute-script-fixes

Conflicts:
src/slonik/slonik.c

updating windows compile docs to mention PGVER

Squashed commit of the following:

commit 0d1c7738dc21f2c9e398b790d101f0024dab0a54
Author: Steve Singer <ssinger@ca.afilias.info>
Date:   Thu Jun 6 10:45:34 2013 -0400

    win32 makefile fixes for the PGVER variable

commit 9f2eed91d129304a0e67534c977394cce04492cf
Merge: c9a4687 86eac22
Author: Steve Singer <ssinger@ca.afilias.info>
Date:   Thu Jun 6 09:42:31 2013 -0400

    Merge remote-tracking branch 'origin/master' into pgcommon_22

commit c9a4687f8fa28cf93a417c0b0597b8c6bf65cea7
Author: Steve Singer <ssinger@ca.afilias.info>
Date:   Sun Jun 2 21:39:23 2013 -0400

    Link with pgcommon on win32 when building with PG 9.3+

commit abd5addee71a2fa55a634673730c2d1d18dfe8e9
Author: Steve Singer <ssinger@ca.afilias.info>
Date:   Sun Jun 2 21:31:37 2013 -0400

    expose HAVE_PGCOMMON to the slonik makefile so it can link in pgcommon if required

    Conflicts:
     configure.ac

commit 9217d72cdf740e7cb9ca7b540c3a7df334f81e0e
Author: Steve Singer <ssinger@ca.afilias.info>
Date:   Tue May 28 10:55:40 2013 -0400

    link with pgcommon in addition to pgport on PG 9.3+

    PG 9.3 introduces a new library called libpgcommon we need to link
    with this when we link with pgport.

uping version/beta number

updating release notes bug 290

versioning all symbols in the shared library.

Slony 2.2 allows multiple versions of the shared library to be exist
on the filesystem($libdir) at once.  This means they can be loaded
into the same backend at once, but we need to ensure that the
exported symbols are unique

Merge branch 'master' into execute-script-fixes

Fix the slonik script scanner to accept two single quotes
inside a single quoted string.

Merge branch 'master' into execute-script-fixes

updating the release notes

When upgrading from an earlier version of 2.2 to a newer version of 2.2
the old shared library (that now has the version number in its name) might
not still be present.  This change avoids having to call C functions as
part of the slonik queries done during a UPDATE FUNCTIONS script.

Fixing several compiler warnings (unused and
possibly used uninitialized things).

Add SQL='string' option to EXECUTE SCRIPT.

Sorting tokens and keywords.

Fixing the magic @STRING@ replacement so that a long line with
such a string crossing a 256 character boundary doesn't fail.

Closing FILE* after reading script files.

Merge remote branch 'origin/master'

Make log_truncate() SECURITY DEFINER.

Make log_truncate() SECURITY DEFINER.

Updating release notes for bug 286

Bug 286 PG 9.3 now takes two arguments on LookupExplicitNamespace

Add a configure check to determine if we have the 1 or 2 argument
version of LookupExplicitNamespace and pass the proper number
of arguments

Merge branch 'master' of ssh://git.postgresql.org/slony1-engine

Add a regression test that ensures DDL is purging the apply cache.

updates to docs and win32 makefile for MS-Windows.
(cherry picked from commit aee311a97a2b11ab4b555943ae0e49fb21c449ff)

Conflicts:

doc/adminguide/installation.sgml

fmgr_info() must be called in the applyCacheContext.

Subsequent calls using the FmgrInfo structure may add more
sub elements to the info. This will be allocated in the same
memory context, the fmgr_info() call was made in. That
context must exist at least long enough as the FmgrInfo
call itself exists.

fixing bug number

updating to release 2.2.0 B3

Bug 285 fix move set race condition.

During a move set it is possible on subscriber node, that isn't the
old or new origin, for remoteWorkerThread_$oldorigin to start
processing a SYNC event before the ACCEPT_SET is processed
by the remoteWorkerThread_$neworigin.  If the ACCEPT_SET
transaction then commits before the update sl_setsync
query is executed by the remoteWorkertThread_$oldorigin
the update will update a row for the new origin when
it intends to update the old origin.  This messes up
sl_setsync.

This change will add a condition to the WHERE clause so
a remoteWorkerThread won't UPDATE rows for a different
origin

updating documentation to list more recent postgresql versions

Add release note material on bugs 273, 282

Merge branch 'bug282'

Merge branch 'bug273'

Bug #284 - fixes to tutorial

Fix for bug273

Do not force the event provider to be part of the providers
unless we don't find any provider at all otherwise.