On Wed, Oct 27, 2004 at 04:21:53PM -0400, Tom Lane wrote: > On the other hand, it's also a pretty minor issue, and if it turns out > to require a lot of code rejiggering to make it do that, I'd not think > it worthwhile.
Patch attached. It passes the regression tests. It shouldn't have secondary effects, but please test. -- Alvaro Herrera (<alvherre[a]dcc.uchile.cl>) "Pensar que el espectro que vemos es ilusorio no lo despoja de espanto, sólo le suma el nuevo terror de la locura" (Perelandra, CSLewis)
Index: src/backend/access/transam/xact.c =================================================================== RCS file: /home/alvherre/cvs/pgsql/src/backend/access/transam/xact.c,v retrieving revision 1.192 diff -c -r1.192 xact.c *** src/backend/access/transam/xact.c 16 Oct 2004 18:57:22 -0000 1.192 --- src/backend/access/transam/xact.c 27 Oct 2004 21:56:21 -0000 *************** *** 2546,2552 **** ereport(WARNING, (errcode(ERRCODE_NO_ACTIVE_SQL_TRANSACTION), errmsg("there is no transaction in progress"))); ! s->blockState = TBLOCK_ABORT_PENDING; break; /* These cases are invalid. */ --- 2546,2553 ---- ereport(WARNING, (errcode(ERRCODE_NO_ACTIVE_SQL_TRANSACTION), errmsg("there is no transaction in progress"))); ! result = true; ! s->blockState = TBLOCK_END; break; /* These cases are invalid. */
---------------------------(end of broadcast)--------------------------- TIP 9: the planner will ignore your desire to choose an index scan if your joining column's datatypes do not match