Fix for bug #138
Fix synatx in autoconf m4 file to define GROFF
(cherry picked from commit b4f202e3f67e66a311a4a32c6392bb2be3bc05a2)

Conflicts:

RELEASE

 check fd for negatvie values. Bug 104 proper fix
(cherry picked from commit 65af4b6002e945c3df977d12b8613b98a379113a)

generated SGML should be ignored by git

Remove reference to logswitch_weekly(), per email from TAKATSUKA Haruka <harukat@sraoss.co.jp> 2010-06-24

Bug #79 - add better documentation about configure options

Making ducttape scripts executable again.

Removed definition of PGAUTODOC. The variable is set in Makefile.global
by configure.

Merge branch 'master', remote branch 'origin'

More removals of references to TABLE ADD KEY

cleanup event function HTML should be ignored

TABLE ADD KEY was still indirectly referenced in the documentation in several places.  Cleaning that out

Merge branch 'master' into bug-146-HEAD

Conflicts:
src/backend/slony1_funcs.sql
tests/test1/generate_dml.sh

Merged HEAD changes

Merge branch 'bug148-head'

Bug 42:

The problem is that postgresql_autodoc produces XML but the documentation is
processed using SGML.  SGML does not process the empty tag notation <xref .../>
correctly.  Instead it should be <xref ...> without slash.  I have attached a
patch that fixes up the XML to be valid SGML so the documentation comes out
right.

Patch by Peter Eisentraut

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

Removing admin guide entry for TABLE ADD KEY since the command
no longer exists.
See bug 147
(cherry picked from commit a2c1cd68c242bc8833b599bb7edb8aef730ed27f)

Bug 147
Remove TABLE ADD KEY from altperl scripts.
Slony-I 2.0.x does not support TABLE ADD KEY so we should not
be generating slonik scripts that try and use it.

Instead generate errors if we find instances where it is required
(cherry picked from commit 9d827a40bf3a1fdaee599628c439870c7d410848)

Conflicts:

RELEASE

Fix for bug #127.
    Generation of syns on leaf nodes (nodes that are not set origins) were disabled some time ago. This reverts
    that commit and re-enables them
(cherry picked from commit 572581c4d80fd95206aed1910fb22d7119aa4be4)

Fix for Bug #120.
    Call enableNode as part of the CLONE NODE processing. This will start the worker thread for the cloned node.
(cherry picked from commit 0d1a293927805e5728c39f8000031bd698ab49c4)

Remove sl_log_1/2 from list of tables to be vacuumed, per comment from Jan in bug #78

Bug #78
http://www.slony.info/bugzilla/show_bug.cgi?id=78

"Christopher Browne 2010-06-02 13:20:53 PDT
We should deprecate the "cleanup_deletelogs" parameter in 2.0.5 or so."

Bug #148
When ShouldSlonyVacuumTable() asks if a table should be vacuumed, the first test should be:
  Does the table exist altogether???
If not, then there's surely no need to vacuum it
'

Bug #142 - Minimize security profile

The two places where superuser access is *truly required* have been
shifted to functions where they could readily be:
 a) Turned into security definers, and
 b) Locked down so only the slony management user(s) can run them

Also added documentation on this to "best practices."

Additional HTML files to be ignored as they are generated from doc tree

Fix for bug # 122

    Print a better error message when using a receiver node id that does not exist
(cherry picked from commit 03db9e0b2c1625aafdaedbf6bf3730c8889cbb2f)

Bug #141 - removing alterTableRestore()

- Applying this to HEAD

Merge branch 'master' of git://git.postgresql.org/git/slony1-engine into bug-146-HEAD

Conflicts:
config.h.in

Initial swing at removing cleanup_deletelogs

Make scripts executable

Changing version number

Make shell scripts executable - expected by tests

Set version number to 2.1.0 for HEAD

Add more generated files to .gitignore

Add FAQ describing recent SSL-related problem experienced by Karl
Denninger

Documentation updates. Mostly typos and minor corrections.
These changes are based on a list of issues submitted by
Tomonari Katsumata, NTT Open Source Software Center

Thanks

Fixed some broken logic in failedNode()

It apparently never supported failing over to nodes that are not
a direct subscriber to the failed node.

Jan
(cherry picked from commit 2705654ec3dcb0381afef591756170c6de8f8ce2)

Conflicts:

src/backend/slony1_funcs.sql

Updating version number to 2.0.4

No longer generate warning of a unsupported version with Pg 9.0
(cherry picked from commit 2d645b3fa1716d649345e8edbb27b188d9505728)

Conflicts:

config.h.in
src/backend/slony1_funcs.sql
src/slonik/slonik.c

Bug #95 - slony.info/bugzilla/show_bug.cgi?id=95

There are two calls to the postgres function subscribeSet_int(integer, integer,
integer, boolean, boolean) in slony1-engine/src/slonik/slonik.c

Both calls do not pass in the last boolean input (the new OMIT COPY parameter)
and hence you get errors.  We have only seen it when using slonik_store_node
but looking at the code it could also happen in slonik_failed_node.
(cherry picked from commit ba90f82da5778648e53d0d742b1fa89901a794e8)

Conflicts:

src/slonik/slonik.c

Add in OMIT COPY option to SUBSCRIBE SET in support of upgrading from
elder Slony-I versions.
(cherry picked from commit dc3f318983213f8fb90eec0545e60dff61f1f55d)

Conflicts:

doc/adminguide/adminscripts.sgml
doc/adminguide/intro.sgml
doc/adminguide/slonyupgrade.sgml
src/backend/slony1_funcs.sql
src/slon/remote_worker.c
src/slonik/parser.y
src/slonik/scan.l
src/slonik/slonik.c
src/slonik/slonik.h
tests/run_test.sh

Perform a va_copy and use the copy in the loop that uses va_args.
You are not allowed to use va_args multiple times.

Fix for bug # 55

Patch submitted by Paul Cannon
(cherry picked from commit f98048352362c990d51a694583bfaa78736cd33b)

Conflicts:

src/slon/misc.c

Updating supportted Pg versions
(cherry picked from commit 1a5b4d6e2df472a7ae0889661ff81f4c8afeb486)

Conflicts:

INSTALL

Improve comment about "missing function" - the problem could be that
libpq wasn't found, not just that you're on some grossly ancient version
of PostgreSQL
(cherry picked from commit 8e510f21fa35358a1b50c5c3c60486c6c56c6fc6)

Remove direct uses of yyleng and instead use the function yyget_leng()
More recent versions of flex have changed yyleng from an int to a size_t.
By using the function we will let the compiler do the conversion if required
on the function result.

We also add an explicit object for scan.o instead of including it at the
bottom of parser.y because we need to generate and include a header for
the scanner (scan.h) so we have the explicit(and correct) definitions of
the yyyget_leng() function.

A autoconf check to ensure that a version of flex with yyget_leng() is
available has also been added.
(cherry picked from commit b44f967e2c62c60f5a2448ce1c4d5afd878922bf)

Conflicts:

RELEASE
configure
configure.ac
src/slonik/Makefile
src/slonik/parser.y
src/slonik/scan.l
src/slonik/slonik.h
src/slony_logshipper/Makefile
src/slony_logshipper/parser.y
src/slony_logshipper/scan.l

Bug 140

The perl module is actually called DBD:Pg not DBI:Pg
Reported by Peter Eisentraut

Fix compile issues on win32
Patch by Hiroshi Saito

Remove all CVS ID tags obsolete in Git

Remove configure, as it is a generated file

Add reference to Bucardo-related tool

Per email from Aleksey Tsalolikhin <atsaloli.tech@gmail.com> on
mailing list.

Revise release checklist to indicate Git-oriented procedures.

Steps that talk about distclean cleaning out .cvsignore have been
removed, as this seems to be nonsense:

 a) It doesn't seem to be actually done in the makefiles
 b) It's certainly obsolete, as we're not using CVS anymore

Revise .cvsignore files --> .gitignore

In many cases, existing .cvsignore files need only be renamed.

In a few cases, new .gitignore files needed to be created as there
wasn't a .cvsignore file (e.g. - for doc/adminguide, src/parsestatements).

Some were missing material (e.g. - src/slon didn't have enough in it).

Revise text about the growth of communications costs to temper wording
that has tended to lead to people saying wild things about Slony-I
performance.

schemadoc.xml is a generated file, so remove it from SCM

Change rules to generate schemadoc.xml automatically, and change "make clean"
to remove it.

fixing output message per report by Gurjeet Singh

Fix script name in help text.

release sl_config_lock before sleeping

See bug # 135, slon appeared to be waiting to get a sl_config_lock
that the same slon process was already holding.
slon kept sleeping waiting for a sync to catch up.
Not holding the lock when we sleep should allow
other threads to obtain the lock.

Merged from 2.0

Log shipping fixes
-
-Proper fix for checking for the proper return code to indicate no files
are queued for processing. This will allow log_shipper to sleep waiting for
more work when no files are ready for processing.  The previous commit was
the wrong commit.

Merged from 2.0

Log shipping fixes
-slony1_dump.sh now generates a .sql file that restores in replica mode
so triggers don't fire.  It will also truncate tables before restoring
to them
-Check for the proper return code to indicate no files are queued for
processing. This will allow log_shipper to sleep waiting for more work
when no files are ready for processing.

Merged from 2.0

In 8.4 and above use TRUNCATE ONLY instead of TRUNCATE when
cleaning out tables before a subscription.  TRUNCATE ONLY
behaves as TRUNCTE did in 8.3.

This fixes bug # 118 - subscribing to a set with an inherited
table  where the inheriting table has a lower tab_id than the
parent table deletes all of the data in the parent table

Merged from 2.0 branch

-Added slonik_add_node altperl script

-Slonik_drop_node now takes an event node

-Changes to the message printed in slon_kill.

Patch by Gurjeet Singh
Merged from 2.0 branch

Fix for bug #43
Use get_set more consistently in the perl tools.
Patch by Peter Eisentraut.

Merged from 2.0

Fix for bug #104
Check for NULL file descriptors before performing actions on them.

Merging from 2.0 STABLE branch

Memory leak fixes submittedb by Ulrich Weber
http://lists.slony.info/pipermail/slony1-hackers/2010-April/000321.html

Merged from REL_2_0_STABLE

Documentation clarification, submitted by singh.gurjeet@gmail.com

Clarifying documentation to be clear that 'DROP SCHEMA' must be
used to drop the slony schema.

The variable name is wrong (syntax error on execution).  Switched to correct
variable name.

Merged from REL_2_0_STABLE branch

Bug # 119

cloneNodeFinish() called updateRelOid() that was looking at tab_id in
sl_sequence but the column is named seq_id

Report/patch from Chirag Dave (cdave@ca.afilias.info)

Merge from REL_2_0_STABLE

Get rid of *some* references to gborg, and point them to new website.

Update supported platforms list, and update mailing list URL

The patch for an issue reported on the mailing list where
tuples with large rows where causing data corruption (reported
as invalid utf8 characters) because the memory was freed while
it was still being used.

This issue was introduced with 1.176.2.8 on Feb 11 2010

Fix shebang, per #108.

Fix sgml error.

Fixed some obsolete prerequisites.

Add notes about bug #55 to test which exercises the fix

SPI_getbinval() was being passed a NULL, which PostgreSQL 8.5 no longer
accepts.  Add isnull variable in the relevant code block so there's a place
to stow the bool result.

Reason for this noted by Alvaro Herrera - thanks!

Port fixes in 2.0 to HEAD...

1.  Inapropos number of parms for insert to confirmations table

2.  Change UPDATEs that break as of 9.0

Bug #106 - change error message for session configuration failure

Apply to HEAD...

Gwyn Connor pointed out a sprintf() abuse...

When log entries are generated, we're passing outbuf both as buffer and as
a source of data.  Behaviour is undefined on some systems, notably including
Gentoo Linux.

Change:  Make sure there are buffers for the date + process, fill them, and
reference them.

"if" should be "else if" as noticed by Richard Yen

Memory leak changes (as made in 2.0) per Ulrich Weber, changed for 2.0/HEAD

http://lists.slony.info/pipermail/slony1-hackers/2010-January/000292.html

Add docs that SERIALTABLES is no longer supported in Slony-I 2.0.

per bug #107

Add notes on dealing with bison/flex mismatch

Bug #110 - some quoting problems

Changed dbutils to use INT64_FORMAT and int64 for handling %L.

- Apply to HEAD

Apply Jan's change from 1.2 branch - buffer overrun

Use int64 rather than long long; that's what PG uses for xids...

As in 2.0 branch, needed to change autoconf + slony1_funcs.c to reflect that
in PG 8.5, ScanKeywordLookup has 3 parms, not 1

Apply fix for bug #103 to HEAD

Fix 8.4-ism as found by Jeff Trout...

In a nutshell in the event loop we start a transaction, then if we are not an accept set event we lock the config lock table.  We then zero out query1. (this is in remote_worker.c).

The ENABLE_SUBSCRIPTION event runs in a while(true) loop.
First it executes query1 (which thanks to the above, is empty), then tries to copy_set.  If copy_set fails for whatever reason we ROLLBACK our local conn (query2) and then loop.

The problem with this is when we come back around in the next look we're outside of a transaction and one won't be started because query1 is reset.  This causes LOCK TABLE to barf on PG8.4.  You are forever stuck until you restart slon.  This also explains another problem I've seen a couple times.

We subscribe to a set with say 3 tables.
The initial subscription fails due to an earlier txn wait.
We copy the first table of hte set successfully.
Then the second table fails to copy due to some DDL issue (perhaps for some reason a PK or column is missing).  We issue a rollback but since we are not in a txn, nothing happens. The event does not suceed so we try again
What happens next is since our previous work wasn't rolled back slony sees we've already got teh deny trigger & friends on the first table and barfs.   Cue infinite loop fixed only by shutting down slon and playing with the sl_ tables.

This patch keeps a count of how many retries we've had on this copy_set.  If we are on retry > 0 then we re-issue a start transaction, set islolation, and lock the config table. My testing has showed that this works.

Bug #98 - need WAIT ON parameter in slonik code generated by slonik_move_set
altperl script.

http://www.slony.info/bugzilla/show_bug.cgi?id=98

insert into sl_setsync upon FAILOVER had values() clause based on the
pre-2.0 schema; fix to use 2.0.

As reported by Cyril Scetbon, bug #97

UPGRADE FUNCTIONS failing because functions had their return types changed.
Added in "DROP FUNCTION IF EXISTS" (fine in 8.3+) in
src/backend/slony1_funcs.sql to rectify this.

Problem reported by Glyn Astill

Add missing arg to subscribeset_int() call - per bug report on list

As observed by Bernd Helmle, cloneNodePrepare returns an event ID, and
hence should return BIGINT

Add 8.3 function to check if tables are handled by autovac, or if slon
should vacuum them itself

Clarify that the EVENT NODE for STORE NODE must be an existing node, not
the newly-added node.

Add $Id$ tag

Add slon_status command.

Usage:

=======================

# slon_status
Usage: slon_status [--config file] node#

    --config file    Location of the slon_tools.conf file

# slon_status 3
Slon is running for the 'replica' cluster.
# slon_status 6
There is no such node.

=======================

Documentation will come later.

Change default $LOGDIR, so that distros won't need to patch it.

Fix init script, so that log directory path will match upstream.