| Commit message (Collapse) | Author | Age |
... | |
|
|
|
|
|
|
|
|
| |
_SPI_execute_plan's return code should reflect the type of the query
that is marked canSetTag, not necessarily the last one in the list.
This is arguably a bug fix, but I'm hesitant to back-patch it because
it's the sort of subtle change that might break someone's code, and it's
best not to do that kind of thing in point releases.
|
|
|
|
|
|
|
| |
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.
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
| |
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.
|
|
|
|
|
|
|
|
| |
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.
|
| |
|
|
|
|
| |
Also fixed a new Coverity report.
|
|
|
|
| |
Further regression cleanup.
|
|
|
|
|
|
| |
_CRT_SECURE_NO_DEPRECATE.
Hiroshi Saito
|
|
|
|
| |
Hiroshi Saito
|
| |
|
|
|
|
|
|
|
|
|
| |
loaded libraries: call functions _PG_init() and _PG_fini() if the library
defines such symbols. Hence we no longer need to specify an initialization
function in preload_libraries: we can assume that the library used the
_PG_init() convention, instead. This removes one source of pilot error
in use of preloaded libraries. Original patch by Ralf Engelschall,
preload_libraries changes by me.
|
|
|
|
| |
Hiroshi Saito
|
| |
|
|
|
|
| |
Lots of small changes in regression test suite
|
|
|
|
|
|
|
|
| |
o print user name for all
o print portal name if defined for all
o print query for all
o reduce log_statement header to single keyword
o print bind parameters as DETAIL if text mode
|
| |
|
|
|
|
|
| |
archiver to behave per original coder's expectation on these machines.
We already know this everywhere else AFAICT.
|
|
|
|
|
|
|
|
|
|
| |
operation every so often. This improves the usefulness of PITR log
shipping for hot standby: formerly, if the standby server crashed, it
was necessary to restart it from the last base backup and replay all
the WAL since then. Now it will only need to reread about the same
amount of WAL as the master server would. The behavior might also
come in handy during a long PITR replay sequence. Simon Riggs,
with some editorialization by Tom Lane.
|
|
|
|
| |
Fixed and cleaned up some regression tests. Also added a new one.
|
|
|
|
|
|
|
|
| |
without indexes) but not to display temp tables. It's a bit hard to
credit that sanity_check could get through a database-wide VACUUM
while the preceding create_index test is still trying to clean up
its temp tables ... but I see no other explanation for the current
failure report from buildfarm member sponge.
|