Merge branch 'bug310'

Bug 311 - Don't include performance.sgml ( 'man') file twice in the tar

bug 309 - make sure sl_failover_targets gets created when upgrading to 2.2

Merge branch 'bug304'

bug 310 - only issue the notify Restart if we actually change configuration

The FAILOVER_NODE event is processed in two parts by slon.
First failedNode(..) is called which reconfigures sl_subscribe,
sl_path and then the listen network. Once this is done notify
is used to signal the slon to restart.

Then the slon commits this transactions and waits for any
pending events from the failed origin to arrive from a third
provider. Slon then continues with the failover (failoverSet_int).

The problem was that when slon restarts it repeats the processing
the FAILOVER_NODE. It was possible for slon to keep restarting
before it gets to be caught up and recording the FAILOVER_NODE
event as processed.

In this patch we only restart slon if the configuration actually
is changed. After the first restart the configuration shouldn't
need changing so there should not be anymore restarts.

deconstruct_array wants the OID of the element type, not the array type

create string_agg in slony1_base.sql not slony1_funcs.sql

If the CREATE AGGREGATE is in slony1_funcs.sql it will fail on an update functions
since the aggregate already exists.  As part of the upgrade we will check
to see if we need to create the aggregate.

Some erroneous queries in date test

Fix typo in sample data

Merge branch 'bug304' of github.com:ssinger/slony1-engine into bug304

Add release note on #304

Revisions to DDL docs

Allow the apply trigger to work (ie not fail) on sl_log_script rows from earlier betas of 2.2.0

bug 304 - record sequence values as part of a EXECUTE_SCRIPT

Prior to 2.2 the EXECUTE_SCRIPT command result in a DDL_SCRIPT event
and a SYNC event generated as part of the same transaction. That
SYNC event also replicates the value of any sequences to the replica
before the DDL SCRIPT command.

In 2.2 the DDL script is replicated as part of a normal SYNC
as rows in sl_log_script.  This commit stores any sequence changes
as extra elements in the cmdargs array.  These sequence values are
then set before the DDL_SCRIPT is executed on the replica

This commit includes a unit test change to reproduce this issue
the test now passes

updating release notes

Merge branch 'bug305'

Fixes to DDL test (old regression)

C requires variable declarations to be at the start of the block

upating win32 symbol definitions for 2.2.0 b5

updating version to for 2.2.0 b5

fixing sytnax errors in documentation SGML

bug 306 release notes update

updating versions for 2.2.0 beta 5

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

set session_replication_role to local in the apply trigger when performing DDL

This fixes bug 305 where user defined (non-replication) triggers don't fire during
the remote/replica exection of EXECUTE_SCRIPT commands

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