This bug looks fixed in 1.3.x code from CVS/SVN; just be sure to to
explicitly declare EVENT NODE in STORE NODE.
Thank you all.
~BAS
On Tue, 27 Mar 2007, Brian A. Seklecki wrote:
In Two places:
line 334 src/slonik/slonik.c:
case STMT_STORE_NODE:
{
SlonikStmt_store_node *stmt =
(SlonikStmt_store_node *) hdr;
if (stmt->ev_origin < 0)
{
stmt->ev_origin = 1;
}
and line 611 src/slonik/parser.y:
stmt_store_node : lno K_STORE K_NODE option_list
{
SlonikStmt_store_node *new;
statement_option opt[] = {
STMT_OPTION_INT( O_ID, -1 ),
STMT_OPTION_STR( O_COMMENT, NULL ),
STMT_OPTION_YN( O_SPOOLNODE, 0 ),
STMT_OPTION_INT( O_EVENT_NODE, 1 ),
STMT_OPTION_END
};
This works for 99.9% of situations, but it may be more pragmatic to use the
node ID of the least number value instead of static "1". E.g., default in
the .y to "-1", and in the .c, to check for "-1" or ...undefined in conf =
default (-1), thus check some algorithm getLowestDefinedNodeID();
The work-around for this is to explicitly declare:
... "event node = [some_node_id_other_than_1_here]"
Which is also really inconsistent because in many other commands this would
be referred to as "node id"; that's a separate issue.
Is this feature request/ticket worthy? Also, are we presently using gborg or
pgfoundry for issue tracking?
TIA, ~BAS
--
l8*
-lava (Brian A. Seklecki - Pittsburgh, PA, USA)
http://www.spiritual-machines.org/
"...from back in the heady days when "helpdesk" meant nothing, "diskquota"
meant everything, and lives could be bought and sold for a couple of pages
of laser printout - and frequently were."
l8*
-lava (Brian A. Seklecki - Pittsburgh, PA, USA)
http://www.spiritual-machines.org/
"Guilty? Yeah. But he knows it. I mean, you're guilty.
You just don't know it. So who's really in jail?"
~James Maynard Keenan
_______________________________________________
Slony1-general mailing list
[email protected]
http://lists.slony.info/mailman/listinfo/slony1-general
