| Commit message (Collapse) | Author | Age |
... | |
|
|
|
|
| |
called replication. Add host and port details, following format
of messages in BackendInitialize().
|
|
|
|
|
|
| |
doing nothing, caused by naptime specified in milliseconds yet units of
pg_usleep() parameter is microseconds. Correctly specifying units
reduces call frequency by 1000. Reduction in CPU consumption verified.
|
|
|
|
|
|
|
| |
and authenticated usernames were swapped. Reported by Bryan Henderson
in bug #5386.
Also clean up poorly-maintained header comment for this function.
|
|
|
|
| |
buildfarm, expend a little more effort on the log message for it.
|
|
|
|
| |
"installcheck-world" make targets as PHONY. Fujii Masao.
|
| |
|
|
|
|
| |
'seconds'.
|
|
|
|
| |
GCC 4.5 complained about it.
|
|
|
|
| |
exclusion-constraint index.
|
|
|
|
| |
exclusion constraints, in line with string length of other pre-9.0 suffixes.
|
|
|
|
|
| |
plain Vars that are generated in the inner rel and used above the join, but
also for PlaceHolderVars. Per report from Oleg K.
|
|
|
|
| |
files instead of relative ones which break vpath builds.
|
|
|
|
| |
regression test that was changed.
|
|
|
|
|
|
|
|
| |
command.
Informix allows variables as argument to the embedded SQL command FREE. Given
that we only allow freeing cursors via FREE for compatibility reasons only we
should do the same.
|
|
|
|
| |
ecpglib.
|
|
|
|
| |
refactored the debug output wasn't adjusted.
|
|
|
|
|
| |
commandline option "-i". This change fixes this and adds a test case. It also
honors #include_next, although this is probably never used for embedded SQL.
|
| |
|
|
|
|
| |
declare section.
|
|
|
|
|
| |
In this case, correction is to remove now unused fields from struct.
Since these were unused and full of garbage anyway, no version change.
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
so that we won't try to attach any context printouts to messages that get
emitted while exiting. Per report from Dennis Koegel, the context functions
won't necessarily work after we've started shutting down the backend, and it
seems possible that debug_query_string could be pointing at freed storage
as well. The context information doesn't seem particularly relevant to
such messages anyway, so there's little lost by suppressing it.
Back-patch to all supported branches. I can only demonstrate a crash with
log_disconnections messages back to 8.1, but the risk seems real in 8.0 and
before anyway.
|
|
|
|
| |
KaiGai Kohei, with adjustments to the comments.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
catalog entries via SearchSysCache and related operations. Although, at the
time that these callbacks are called by elog.c, we have not officially aborted
the current transaction, it still seems rather risky to initiate any new
catalog fetches. In all these cases the needed information is readily
available in the caller and so it's just a matter of a bit of extra notation
to pass it to the callback.
Per crash report from Dennis Koegel. I've concluded that the real fix for
his problem is to clear the error context stack at entry to proc_exit, but
it still seems like a good idea to make the callbacks a bit less fragile
for other cases.
Backpatch to 8.4. We could go further back, but the patch doesn't apply
cleanly. In the absence of proof that this fixes something and isn't just
paranoia, I'm not going to expend the effort.
|
|
|
|
|
| |
in the xact field on replay, due to not writing out all the data in
the wal log struct.
|
|
|
|
|
|
|
| |
was broken for a replication connection and no messages were
displayed on either standby or primary, at any debug level.
Connection messages needed to diagnose session drop/reconnect
events. Use LOG mode for now, discuss lowering in later releases.
|
|
|
|
|
| |
correctly sized and expand comment to explain otherwise
undocumented use of replication connection parameter.
|
| |
|
|
|
|
|
|
|
|
| |
present since 8.0 was never fully meaningful, since two recovery targets
cannot be specified. Refactor recovery target type to make this change
and associated code easier to understand. No change in function.
Bug report arising from internal support question.
|
|
|
|
|
|
|
| |
field into WAL record and reset it from there, rather than using
FrozenTransactionId which can lead to some corner case bugs.
Problem report and suggested route to a fix from Heikki, details by me.
|
| |
|
|
|
|
| |
with a few strategically placed pg_verifymbstr calls.
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
| |
In PLy_spi_execute_plan, use the data-type specific Python-to-PostgreSQL
conversion function instead of passing everything through InputFunctionCall
as a string. The equivalent fix was already done months ago for function
parameters and return values, but this other gateway between Python and
PostgreSQL was apparently forgotten. As a result, data types that need
special treatment, such as bytea, would misbehave when used with
plpy.execute.
|
|
|
|
|
|
|
|
|
| |
in recovery_end_command, it always came out as 0 because InRedo was
cleared before recovery_end_command was executed. Also, always take
ControlFileLock when reading checkpoint location for %r.
The recovery_end_command bug and the missing locking was present in 8.4
as well, that part of this patch will be backported separately.
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
to transformAggregateCall, instead of abusing fields in Aggref to carry them
temporarily. No change in functionality but hopefully the code is a bit
clearer now. Per gripe from Gokulakannan Somasundaram.
|
|
|
|
|
|
| |
the master is still in recovery. We don't support cascading slaves yet.
Patch by Fujii Masao, with slightly changed wording.
|
|
|
|
| |
correct, as described in comments at start of xlog.c
|
| |
|
| |
|
| |
|
|
|
|
| |
Jaime Casanova
|
|
|
|
| |
Add ERRCODE_INVALID_PASSWORD sqlstate error code.
|
|
|
|
|
|
| |
by Dan O'Hara.
Patch by Teodor Sigaev
|
| |
|
|
|
|
|
|
|
|
|
|
| |
unable to read a stats file for reasons other than ENOENT, and having to reset
last_statrequest because it's later than current time in the collector.
Not clear if this will shed any light on the "pgstat wait timeout" business,
but it seems like a good idea in general.
In passing, do some message-style-police work on recently-added
pgstat_reset_shared_counters code.
|
|
|
|
| |
properly; it's been making that comment uglier with each run.
|