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

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

Fix for Bug #120.
    Call enableNode as part of the CLONE NODE processing. This will start the worker thread for the cloned node.

Updating release notes with recent commits

Fix for bug # 122

    Print a better error message when using a receiver node id that does not exist

Make scripts executable, as in commit 205e849c10d41fffac24f1401cfaa9507b7dd03c

Bug #141 - removing alterTableRestore(), obsolete as of 2.0

add generated scan.h files to .gitignore

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

commit e86d296ae8c739018100306bcebeded80525f265 seems to have
been a merge or replace of the master branch into REL_2_0_STABLE.

This is not what should have been intended as it erased numerous changes
that were only in the 2.0 branch that should be merged into master.

This commit will put those changes back into 2.0, it undoes parts of
e86d29 but leaves the CVS tags removed.

This commit also undoes some changes made in master (mostly documetnation)
that maybe should go in 2.0. If that is the case they will need to be
put back as seperate commits

This commit was prepared by going back to before e86d29 and redoing the
tag removal and .gitignore changes then merging in the other changes done to
REL_2_0_STABLE after.  This patche was then prepared to make the current
REL_2_0_STABLE look like the result.

Bug 140

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

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

Put in empty <!--  --> and /*   */ to make 2.0 consistent with the
other branches

Fix compile issues on win32
Patch by Hiroshi Saito

Chris evidently didn't quite handle $Id$ removal correctly for 2.0
branch.  Removing /* and <-- lines that no longer have $Id$ tags.

Ancient README that references a wiki that Chris had installed several
years ago on a firewall box that is now long dead

Remove all CVS ID tags obsolete in Git

fixing output message per report by Gurjeet Singh

Fix script name in help text.

updating release notes

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.

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.

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.

Updating release notes for bug #118

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

-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

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

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

Removing extra characters at bottom of file

Updating supportted Pg versions

Updating release notes

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

Updating version number to 2.0.4

No longer generate warning of a unsupported version with Pg 9.0

checkin in a regenerated configure, the previous commit didn't reflect the cvs version of some m4 files.  We really shouldn't have configure scripts in cvs

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.

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

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)

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

Update mailing list URL, and also add Lenny to supported platforms list.

Update supported platforms list.

Jan's 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

Documentation fix: a 'wait for event' following a 'lock set' does not
actually work(no event to wait on).  You do not need to wait for the lock set
because once the 'lock set' returns you already have the lock.

Fix shebang, per #108.

Fix sgml error.

Changing label to match 2_0_3 instead of the generic 'stable'

Commit configure, after building it

Lots of updates to release notes preparatory to 2.0.3 RC4

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

DROP cloneNodePrepare if it exists because the return type might have changed.
Bug # 113 patch from Atte Peltomäki

Fixed some obsolete prerequisites.

Add notes about bug#55 to release notes, regression test

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

slony1_extract_schema.sh will now compare the major versions of postgres and pg_dump
to make sure they match.

Added slony1_extract_schema_for_upgrade.sh a wrapper script that will handle
the requried steps for getting a schema from an older postgres that can be
used as input to a newer version

Addresses bug # 114, merged from 1.2 branch

file slony1_extract_for_upgrade.sh was added on branch REL_2_0_STABLE on 2010-03-30 18:05:17 +0000

Updating comment to reflect reality.
+ The below log message that should have been applied with the previous checkin
-------
Load the v84 slony1_funcs file when running on an 8.4 system.
This checks the autovacuum status in the proper way for 8.4

Bug 116
Patch submitted by Peter Eisentraut

msg

Add notes on recommending *not* suppressing WAL via synchronous_commit
GUC per discussion on list with Peter Geoghegan

Update release notes

Improve error reporting to indicate DB conn info in case where failure is
due to bad conn info

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!

Found a query that no longer runs in 9.0 because subqueries in updates
evidently can't reference the "parent" table

More revisions to release notes based on recent fixes

Bug #106 - change error message for session configuration failure

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.

Add release notes for recent changes

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

Memory leak in remote worker, as found by Ulrich Weber

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

Note that in 2.0, code has changed so that only some of the changes are
applicable

Bison/Flex FAQ addition as in HEAD

Bug #110 - some quoting problems

Remove Emacs fodder from test scripts - bug #108

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

- Apply to 2.0

Apply Jan's buffer overflow fix to 2.0 branch

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

Capture autoconf change in release notes

Add an 8.5-ism...

ScanKeywordLookup now expects 3 arguments.

Added an autoconf test to check to see if there's 1 or 3 arguments

Changed src/backend/slony1_funcs.c to use 1/3 arguments, as appropriate

Add bug #103 to release notes

Bug #103: bugs.slony.info/bugzilla/show_bug.cgi?id=103

As observed by Cyril Scetbon, we need to initialize snapshot values to
"1:1:", not "0:0:", as the latter is refused by the snapshot parser.

Release note updates (for 2.0.3 rc3)

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.

Bug #102

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

If you have a configuration with 1 master and 2 or more slaves, and you have
all paths defined, slonik can hang during a failover.

In src/slonik/slonik.c, the slonik_failed_node function queries the sl_nodelock
table on each node to find the listener process responsible for that node and
stores it in nodeinfo.

Later in the function it loops through all the nodes, checking to see if the
listener responsible for that node has exited so it knows that slon has
restarted on that node.

Unfortunately, the query it uses just counts the number of processes that are
not that original pid. It obviously is expecting only one entry in the result
set (i.e., the replacement listener). If there are other listeners for other
nodes running on that node (as is the case when a second slave has a path
defined, for example), then that query may never return exactly 1.

The fix is to add the node number to the query, since then it finds the number
of processes that aren't the old listener that are assigned to that node. When
there is exactly 1 of those then the slon has restarted.

Per Michael Lee Squires @ whitepages.com

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

Typo: column referred to tab_reloid when it should refer to seq_reloid

As reported by Shaun Jackson

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.

Missed a fix to insert into sl_setsync; per Cyril Scetbon

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 in 8.3 function to check what tables should be left unvacuumed
(because autovac covers this)

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

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

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

Update copyrights

Update copyrights

Fix LOG_NAME_SUFFIX feature. Thanks to David Fetter and Andrew Dunstan.

Functions that return event IDs need to be returning "bigint", not "int4"
or "integer"

Change numerous cases where integer values are appended onto strings to
specifically cast to text.  This addresses the issue where, in 8.3+,
casts aren't done as much implicitly