| Commit message (Collapse) | Author | Age |
... | |
|
|
|
|
|
|
| |
a Coverity warning, these are risky since the hashtable isn't necessarily
fully set up yet. They're unnecessary anyway: a deletable hashtable
should be in a memory context that will be cleared following elog(ERROR).
Per report from Martijn van Oosterhout.
|
|
|
|
| |
values, due to concern about the patch.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
and instead make the grammar production for the RETURN statement do the
heavy lifting. The lookahead idea was copied from the main parser, but
it does not work in plpgsql's parser because here gram.y looks explicitly
at the scanner's yytext variable, which will be out of sync after a
failed lookahead step. A minimal example is
create or replace function foo() returns void language plpgsql as '
begin
perform return foo bar;
end';
which can be seen by testing to deliver "foo foo bar" to the main parser
instead of the expected "return foo bar". This isn't a huge bug since
RETURN is not found in the main grammar, but it could bite someone who
tried to use "return" as an identifier.
Back-patch to 8.1. Bug exists further back, but HEAD patch doesn't apply
cleanly, and given the lack of field complaints it doesn't seem worth
the effort to develop adjusted patches.
|
|
|
|
|
|
| |
when what's being executed is a COMMIT or ROLLBACK. Per report from
Sergey Koposov. Backpatch to 8.1; 8.0 and before don't have the bug
due to lack of any logging at all here.
|
|
|
|
| |
for INSERT/UPDATE/DELETE RETURNING. Per discussion.
|
|
|
|
|
| |
so that it won't interfere with later trials. Per recent buildfarm
experience. Anyone know how to do this on Windows?
|
|
|
|
|
|
|
|
| |
well as vacuum_cost_delay. Since datestyle is a string variable,
this exercises memory allocation issues that might not appear when
modifying an integer GUC variable. Also, we can observe the side
effects of changing datestyle to check that assign hooks are called
at the right times.
|
|
|
|
|
|
| |
nonunique join value, leading to plan-choice-dependent results ... and
it seems some platforms will choose a different plan. Tweak the test
so that it has well-defined results. Per report from Olivier Prenant.
|
| |
|
| |
|
|
|
|
|
| |
Merged dyntest.pgc and dyntest2.pgc.
Hopefully fixed the last Coverity reports (finally)
|
|
|
|
| |
hash and array variables. (regression output updated)
|
|
|
|
| |
default values.
|
| |
|
|
|
|
|
|
|
|
| |
merely a matter of fixing the error check, since the underlying Portal
infrastructure already handles it. This in turn allows these statements
to be used in some existing plpgsql and plperl contexts, such as a
plpgsql FOR loop. Also, do some marginal code cleanup in places that
were being sloppy about distinguishing SELECT from SELECT INTO.
|
| |
|
|
|
|
| |
hashes. Was causing regression failures.
|
|
|
|
| |
failures.
|
|
|
|
| |
default values. Was causing regression failures.
|
|
|
|
|
|
|
|
|
|
| |
< o Allow INSERT/UPDATE ... RETURNING new.col or old.col
<
< This is useful for returning the auto-generated key for an INSERT.
< One complication is how to handle rules that run as part of
< the insert.
< http://archives.postgresql.org/pgsql-patches/2005-07/msg00568.php
> o -Allow INSERT/UPDATE ... RETURNING new.col or old.col
|
| |
|
|
|
|
|
|
|
|
| |
plpgsql support to come later. Along the way, convert execMain's
SELECT INTO support into a DestReceiver, in order to eliminate some ugly
special cases.
Jonah Harris and Tom Lane
|
|
|
|
| |
Yoshiyuki Asaba
|
|
|
|
|
| |
< o %Allow commenting of variables in postgresql.conf to restore them
> o -Allow commenting of variables in postgresql.conf to restore them
|
| |
|
|
|
|
|
|
| |
defaults.
Zdenek Kotala
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The main reason for refactoring was that set_config_option() was too
overloaded function and its behavior did not consistent. Old version of
set_config_function hides some messages. For example if you type:
tcp_port = 5432.1
then old implementation ignore this error without any message to log
file in the signal context (configuration reload). Main problem was that
semantic analysis of postgresql.conf is not perform in the
ProcessConfigFile function, but in the set_config_options *after*
context check. This skipped check for variables with PG_POSTMASTER
context. There was request from Joachim Wieland to add more messages
about ignored changes in the config file as well.
Zdenek Kotala
|
|
|
|
|
|
|
| |
Allow conversion from perl to postgresql array in OUT parameters. Second,
allow hash form output from procedures with one OUT argument.
Pavel Stehule
|
|
|
|
| |
David Fetter
|
| |
|
| |
|
|
|
|
| |
Ian Barwick
|
|
|
|
|
|
|
| |
< * Allow encoding on a per-column basis
> * Allow encoding on a per-column basis optionally using the ICU library:
> http://archives.postgresql.org/pgsql-patches/2005-08/msg00309.php
> http://archives.postgresql.org/pgsql-patches/2006-03/msg00233.php
|
|
|
|
|
|
|
| |
same data type and same typmod, we show that typmod as the output
typmod, rather than generic -1. This responds to several complaints
over the past few years about UNIONs unexpectedly dropping length or
precision info.
|
|
|
|
| |
comment. 8.1.X is not affected by this commit.
|
|
|
|
|
|
| |
interface builds like libpq need it.
Backpatch addition to 8.1.X.
|
|
|
|
| |
Greg Sabino Mullane
|
|
|
|
| |
will fix current platypus build failure.
|
| |
|
|
|
|
|
|
| |
rounded down to zero.
Backpatch to 8.1.X.
|
|
|
|
| |
> http://archives.postgresql.org/pgsql-hackers/2006-08/msg00149.php
|
|
|
|
|
|
|
|
| |
as micro-seconds, rather than as 100 microseconds, as it does now. This
actually fixes all setitimer calls on Win32, but statement_timeout is
the most visible fix.
Backpatch to 8.1.X. 8.0 works as documented.
|
| |
|
|
|
|
| |
> http://archives.postgresql.org/pgsql-performance/2004-08/msg00349.php
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
>
>
> Features We Do _Not_ Want
> =========================
>
> * All backends running as threads in a single process (not want)
>
> This eliminates the process protection we get from the current setup.
> Thread creation is usually the same overhead as process creation on
> modern systems, so it seems unwise to use a pure threaded model.
>
> * Optimizer hints (not want)
>
> Optimizer hints are used to work around problems in the optimizer. We
> would rather have the problems reported and fixed.
> http://archives.postgresql.org/pgsql-hackers/2006-08/msg00506.php
|
|
|
|
|
|
| |
< A package would be a schema with public/private variables,
> A package would be a schema with session-local variables,
> http://archives.postgresql.org/pgsql-hackers/2006-08/msg00384.php
|
|
|
|
| |
Also fixed a new Coverity report.
|
|
|
|
| |
Further regression cleanup.
|
|
|
|
|
|
| |
_CRT_SECURE_NO_DEPRECATE.
Hiroshi Saito
|
|
|
|
|
|
|
| |
< A package would be a schema with its own variables,
< private functions, and initialization functions. It
> A package would be a schema with public/private variables,
> public/private functions, and initialization functions. It
|