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

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.

Fix for bug282.

There was a problem with the new COPY protocol when
a data type or domain used in a replicated column
is not in slon's search_path. SPI does not provide
a mechanism to get the namespace name of a columns
data type.

Instead of adding explicit type casting to the apply
queries and handing the data in as TEXT Datums, we
now use the same technique that PL/pgSQL uses at
at least since 8.3 and convert the TEXT datums into
the requested data type ourselves.

additional cleanups for pgport find_my_exec support

Clean up configure messaging and avoid a compiler warning by
checking for HAVE_PGPORT after config.h is included
(cherry picked from commit be933801d527caa7c88df622b6ec351668189281)

Conflicts:

RELEASE
src/slonik/slonik.c

Add back in find_my_exec() support on platforms that include pgport.
find_my_exec, a postgresql pgport function is again used to find the
location of the slonik binary to locate the share directory.
This is for installs where the install location is not known at
compile time.

Some distributions don't include pgport, this is disabled by default
if you compile with pgport support set to yes (a configure option/check)
then this will be enabled.

Patch submitted by Sandeep Thakkar <sandeep.thakkar@enterprisedb.com>
(cherry picked from commit b034434e6279da67008402575ed768a666b827a9)

Conflicts:

configure.ac

Add in code to recognize PostgreSQL 9.2 and accept it as a
supported version, rather than warning that we're not sure

Conflicts:
RELEASE
src/slonik/slonik.c

adding altperl fixes to the release notes

Merge remote branch 'lkmatsumura/altperl_master'

updating release to 2.2.0 b2

Cosmetic commit - trailling spaces

removing trailling spaces

Bug fix, incorrect variable name

temporary pgpass file without password because of this bug

get_pid() will check to make sure the pid actually exists

include pidfile directory in the sample slon.conf

add --only-node option to usage message

set the PGPASSWORD file to read-only, then close and unlink it

Implementing pidfile to identify the main process of slony instance:

slon_tools.conf-sample - add configuration variables
  . $PIDFILE_DIR : directory where the pid files will be created (default: /var/run/slony1)
  . $PIDFILE_PREFIX : prefix to pidfiles of node instances (default: cluster name)

slon-tools.pm:
 . start_slon : changed to init slon daemon with -p pidfile and added a sleep time
    to give time to slon daemon initialize and create the pid file

 . get_pid: changed to read the pid file to get the pid number

slon_kill.pl:
  . change kill method of slon daemon to get_pid then use kill 15 instead 9 to
    slon daemon remove the pid file properly

slon_watchdog2.pl:
  . Add sleep time before first query status to avoid the watchdog kill slon_daemon
    just immediatelly they started
  . Do not try to kill slon proccess with 9.

Using PGPASSFILE environment variable instead of PGPASSWORD

Using a temporary pgpass file to store password as recommended by Postgresql documentation

Thanks to Steve Singer review

Init script sample using altperl routines

Example of a start/stop service script using altperl and a
slon config file example

slony_show_configuration can now return cluster name property

New Watchdog options - verbosity

Implementing new parameters on slon_tools.conf
WATCHDOG_VERSION : select watchdog to start
   1: slon_watchdog
   2: slon_watchdog2

WATCHDOG_VERBOSE: enable/disable verbosity of watchdog logs

START_WATCHDOG and SLEEP_TIME: just declaring since the funcionality
  was already implemented

slon_watchdog2: making the routine functional since some bugs was detected,
  routine updating as changes detected on slon_watchdog

Implementing capability of kill only a node on slon_kill

New parameter to kill only processes of one node
Refactoring to create subroutines.
Adjust of commands to grant that they will be sorted by pid values

Change get_pid to return the smallest pid

get_pid changed to return the smallest pid of processes that match
the regex, probably the main proccess