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

Update "date"

Update supported platforms list. I believe more can be added by Chris (AIX).

On 8.4, pg_class.reltriggers no longer exists, so query that updates this
should check for existence first

Diminish some logging from INFO to DEBUG1

Per suggestion from Jeff Frost

Update release notes for recent changes

Aleksander Kmetec submitted this patch which addresses a race condition
potentially effecting long time locking or data loss on TRUNCATE.

Dataloss bug, described here:
http://lists.slony.info/pipermail/slony1-general/2009-July/009663.html

Statement blocking bug, described here:
http://lists.slony.info/pipermail/slony1-general/2009-July/009664.html

Patch is to preface TRUNCATE logic with an attempt (in an exception
block) to acquire an exclusive lock on the apropos sl_log_* table.
The NOWAIT option raises a (caught!) exception if there are any other
processes holding onto the table.

Per Devrim Gunduz...

Add a "LOG_NAME_SUFFIX" variable to slon-tools to allow centrally
configurating the filename for log files

Revise release notes for autovac changes

Oops, a widespread typo in the testbed which was hidden because the
defaults "happened to work" (until I started 8.4-based testing!)

Addressing autovacuum change in PostgreSQL 8.4

1.  Need to have v84.sql files

2.  Need for slonik to install v84.sql files, when apropos

3.  v84.sql functions file implements alternative implementation
    which pulls the attribute from pg_class rather than the former
    pg_autovacuum table

Introduce "%L" which is interpreted as "%lld" - bug #92

Add release notes for recent changes

Change actionseq values from "int" to "long long" to accomodate large
sequence values - per bug #92

Change minimum logging level to -1, as requested

Per Mark Stosberg, need to have some docs for altperl script "slonik_drop_sequence"

1.  Mark 2.0.3 version, preparatory to release

2.  Update release notes to include changes (to date) to 2.0.3

3.  slonik.c now recognizes PostgreSQL 8.4 properly

Need extra "*cp++ = c;" in 4 places in logshipping scanner.

Per Richard Yen, on mailing list

Cleanup of slonikconfdump.sh
 - There were a couple bugs
 - Variables were quite badly named making it rather opaque

Add in OMIT COPY option to SUBSCRIBE SET in support of upgrading from
elder Slony-I versions.

Revised slonik dump script based on input from our DBA group...
It now uses tsort to determine the subscription ordering, so should
work even with fairly sophisticated cascaded subscriptions

Add OMIT COPY test to 2.0

Per Jan's comments, this is a valid change; it also needs to be applied
to the 2.0 branch

Change ordering of subscription extraction, and document the restrictions
a bit more carefully for the configuration dumper

Add in a slonik configuration dump tool, that will be helpful when doing
upgrades from 1.2 to 2.0, along with documentation.

Update sequence test to validate that big ID #'s do not cause
grief

Fix sgml error

Fix problem reported by Melvin Davidson relating to cleanup thread.

VACUUM/ANALYZE request return codes were being misinterpreted as errors.

Fixed this, so that:
 - Errors are reported as such
 - Warnings are reported as such

Also modified partitioning test to suppress autovac on at least one table
so that regression tests may be expected to exercise this code.

Fix problem with interpretation of return code for VACUUM/ANALYZE

Per Melvin Davidson

If PGSHARE is provided as part of the environment, then we should use it

Add note that if syslog is turned on, then timestamp generation is
suppressed.

Per bug #91, some config parameters were not documented in the sample
slon.conf file