Geoffrey <lists at serioustechnology.com> writes:
> Christopher Browne wrote:
>
>> I'm seeing some agreement on the merits of this; I'll hold off on the
>> change until I see one or two more "+1"'s :-).
>
> +1

I have gotten enough of a population of "+1"s, including Jan
indicating that he thought it was a mistake to have default values of
1's in the first place.

I have committed the change to CVS HEAD.  Alas, it got split across
about 4 commit messages, so I can't just point to one :-(.

----------------------------------------------------------------------
Various instances where slonik would use a default node ID of 1 have been changed to remove this.

Slonik scripts may need to be changed to indicate an EVENT NODE (or
similar) after migration to v2.0 as a result.

The slonik commands involved:

- STORE NODE - EVENT NODE
- DROP NODE - EVENT NODE
- WAIT FOR EVENT - WAIT ON
- FAILOVER - BACKUP NODE
- EXECUTE SCRIPT - EVENT NODE
----------------------------------------------------------------------

Things updated include:
 a) slonik sources
 b) admin guide
 c) tests under tests/
 d) I'm about to commit a fix to tools/configure-replication.sh
 e) I'm taking a walk thru the ducttape tests; found a bunch of changes to make :-(
-- 
let name="cbbrowne" and tld="ca.afilias.info" in name ^ "@" ^ tld;;
<http://dba2.int.libertyrms.com/>
Christopher Browne
(416) 673-4124 (land)