This is a discussion on bug fix - plperl %_SHARED misspelled within the Pgsql Patches forums, part of the PostgreSQL category; --> While building some better plperl tests today I discovered a horrid bug (which I regret to say is my ...
| |||||||
| FAQ | Members List | Calendar | Search | Today's Posts | Mark Forums Read |
| ||||
| While building some better plperl tests today I discovered a horrid bug (which I regret to say is my fault), present in both 8.0 and HEAD branches, The attached patch needs to be applied urgently to both branches, please. cheers andrew Index: plperl.c ================================================== ================= RCS file: /projects/cvsroot/pgsql/src/pl/plperl/plperl.c,v retrieving revision 1.73 diff -c -r1.73 plperl.c *** plperl.c 6 May 2005 17:24:55 -0000 1.73 --- plperl.c 21 May 2005 20:56:02 -0000 *************** *** 226,232 **** "$PLContainer->permit_only(':default');" "$PLContainer->permit(qw[:base_math !:base_io sort time]);" "$PLContainer->share(qw[&elog &spi_exec_query &DEBUG &LOG " ! "&INFO &NOTICE &WARNING &ERROR %SHARED ]);" "sub ::mksafefunc { return $PLContainer->reval(qq[sub { $_[0] $_[1]}]); }" ; --- 226,232 ---- "$PLContainer->permit_only(':default');" "$PLContainer->permit(qw[:base_math !:base_io sort time]);" "$PLContainer->share(qw[&elog &spi_exec_query &DEBUG &LOG " ! "&INFO &NOTICE &WARNING &ERROR %_SHARED ]);" "sub ::mksafefunc { return $PLContainer->reval(qq[sub { $_[0] $_[1]}]); }" ; ---------------------------(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 |
| |||
| Andrew Dunstan wrote: > While building some better plperl tests today I discovered a horrid bug > (which I regret to say is my fault), present in both 8.0 and HEAD > branches BTW, in future it would be good to specify the consequences of the bug (i.e. beyond "it is horrible"), so committers who don't use pl/perl can judge the urgency of the bug. > The attached patch needs to be applied urgently to both > branches, please. Applied, thanks. -Neil ---------------------------(end of broadcast)--------------------------- TIP 4: Don't 'kill -9' the postmaster |
| ||||
| Neil Conway said: > > BTW, in future it would be good to specify the consequences of the bug > (i.e. beyond "it is horrible"), so committers who don't use pl/perl can > judge the urgency of the bug. Yes, sorry. In fact, fortuitously the effects are not disastrous in most circumstances. Once we turn on strict mode they would become severe - %_SHARED would be unusable. But fixing the error has zero downside impact. cheers andrew ---------------------------(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 |