From: Tom Lane t...@sss.pgh.pa.us
In the first query, the MemoryContextReset is nearly free since there's
nothing to free (and we've special-cased that path in aset.c). It's
certainly a measurement artifact that it measures out faster, which says
to me that these numbers can only be trusted
Hello,
Before continueing discussion, I tried this patch on the current
head.
This patch applies cleanly except one hunk because of a
modification just AFTER it, which did no harm. Finally all
regression tests suceeded.
Attached is the rebased patch of v11 up to the current master.
regards,
Hi Sawada Masahiko,
I liked this feature. So I have reviewed it.
Changes are straight forward and looks perfect.
No issues found with make/make install/initdb/regression.
However I would suggest removing un-necessary braces at if, as we have only
one statement into it.
if (++cur_line =
On Thu, Jun 12, 2014 at 10:55 PM, Robert Haas robertmh...@gmail.com wrote:
On Wed, Jun 11, 2014 at 7:42 AM, Magnus Hagander mag...@hagander.net wrote:
Replication commands like IDENTIFY_COMMAND are not logged even when
log_statements is set to all. Some users who use log_statements to
audit
Re: Tom Lane 2014-06-18 21034.1403110...@sss.pgh.pa.us
Christoph Berg christoph.b...@credativ.de writes:
now that we have vacuumdb --all --analyze-in-stages in 9.4, wouldn't
it make sense to get rid of the analyze_new_cluster.sh file which
pg_upgrade writes? The net content is a single line
On Wed, Jun 18, 2014 at 12:50 PM, Shreesha shreesha1...@gmail.com wrote:
Well, the initdb issue looks to be resolved. Actually, after making the
changes as suggested by Kyotaro Horiguchi, I copied only initdb binary to my
platform and didn't copy all of them. Hence, the dependencies were still
On Fri, Jun 13, 2014 at 11:29 PM, Tom Lane t...@sss.pgh.pa.us wrote:
Fujii Masao masao.fu...@gmail.com writes:
On Thu, Jun 12, 2014 at 8:51 PM, Fujii Masao masao.fu...@gmail.com wrote:
Some users enable log_disconnections in postgresql.conf to audit all
logouts.
But since log_disconnections
Fujii Masao masao.fu...@gmail.com writes:
OK, I've just implemented the patch (attached) which does this, i.e.,
redefines
log_statement as a list. Thanks to the patch, log_statement can be set
to none,
ddl, mod, dml, all, and any combinations of them. The meanings of
none, ddl, mod and all
On Thu, May 8, 2014 at 10:00 AM, Jaime Casanova ja...@2ndquadrant.com wrote:
Hi,
This patch implements $subject only when ANALYZE and VERBOSE are on.
I made it that way because for years nobody seemed interested in this
info (at least no one did it) so i decided that maybe is to much
On Fri, Jun 20, 2014 at 10:48 PM, Tom Lane t...@sss.pgh.pa.us wrote:
Fujii Masao masao.fu...@gmail.com writes:
OK, I've just implemented the patch (attached) which does this, i.e.,
redefines
log_statement as a list. Thanks to the patch, log_statement can be set
to none,
ddl, mod, dml, all,
On Wed, Jun 18, 2014 at 8:41 AM, Christoph Berg
christoph.b...@credativ.de wrote:
Hi,
now that we have vacuumdb --all --analyze-in-stages in 9.4, wouldn't
it make sense to get rid of the analyze_new_cluster.sh file which
pg_upgrade writes? The net content is a single line which could as
well
Re: Jeff Janes 2014-06-20
CAMkU=1z3Edq+CNRo4F=jBEzXNMidSskdm=cpcaznogdy2si...@mail.gmail.com
On Wed, Jun 18, 2014 at 8:41 AM, Christoph Berg
christoph.b...@credativ.de wrote:
Hi,
now that we have vacuumdb --all --analyze-in-stages in 9.4, wouldn't
it make sense to get rid of the
Hi,
When creating postgis extension, a call to DefineCustomStringVariable is
made to define postgis.backend.
An assign hook allows to change the library used by spatial functions
whith a:
SET postgis.backend = 'sfcgal'; -- or 'geos'
Everything work fine execept when upgrading postgis.
Vincent Mora vincent.m...@oslandia.com writes:
If I understand the problem correctly, on upgrate (ALTER EXTENSION), the
old postgis library is already loaded, the old spatial functions are
replaced by new ones pointing to the new postgis library, a call to
DefineCustomStringVariable is made
Some of my Salesforce colleagues are looking into making every system
catalog be declared with a true primary key. They came across the
fact that pg_seclabel and pg_shseclabel are declared with unique
indexes that include the provider column, but that column does not
get marked as NOT NULL during
On 06/20/2014 11:26 AM, Joey Caughey wrote:
I’m having an issue with JSON requests in Postgres and was wondering
if anyone had an answer.
I have an orders table with a field called “json_data”.
In the json data there is a plan’s array with an id value in them.
{ plan”: { “id”: “1” } } }
I
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 06/19/2014 03:38 PM, MauMau wrote:
From: Joe Conway m...@joeconway.com
Fair enough -- this patch does it at that level in
materializeQueryResult()
I'm in favor of this. TBH, I did this at first, but I was afraid
this would be rejected due
On Jun 20, 2014, at 10:01 AM, Tom Lane t...@sss.pgh.pa.us wrote:
Some of my Salesforce colleagues are looking into making every system
catalog be declared with a true primary key. They came across the
fact that pg_seclabel and pg_shseclabel are declared with unique
indexes that include the
Robert Haas robertmh...@gmail.com writes:
On Jun 20, 2014, at 10:01 AM, Tom Lane t...@sss.pgh.pa.us wrote:
Some of my Salesforce colleagues are looking into making every system
catalog be declared with a true primary key. They came across the
fact that pg_seclabel and pg_shseclabel are
Tom Lane wrote:
The idea I'm toying with right now is to additionally mark as not nullable
any column referenced in a DECLARE_UNIQUE_INDEX command in
catalog/indexing.h. But I've not looked through that set carefully; it's
conceivable that we actually have some indexed catalog columns that
On 2014-06-20 16:50:15 -0400, Alvaro Herrera wrote:
Tom Lane wrote:
The idea I'm toying with right now is to additionally mark as not nullable
any column referenced in a DECLARE_UNIQUE_INDEX command in
catalog/indexing.h. But I've not looked through that set carefully; it's
conceivable
Andres Freund and...@2ndquadrant.com writes:
On 2014-06-20 16:50:15 -0400, Alvaro Herrera wrote:
Non-unique indexes in indexing.h
already bear a standard comment that they are not used for syscache.
The only exception was added recently in f01d1ae3a104019:
On 2014-06-20 17:29:33 -0400, Tom Lane wrote:
Andres Freund and...@2ndquadrant.com writes:
On 2014-06-20 16:50:15 -0400, Alvaro Herrera wrote:
Non-unique indexes in indexing.h
already bear a standard comment that they are not used for syscache.
The only exception was added recently in
Jeff Janes wrote:
12538 2014-06-17 12:10:02.925 PDT:LOG: JJ deleting 0xb66b20 5183
12498 UPDATE 2014-06-17 12:10:03.188 PDT:DETAIL: Could not open file
pg_multixact/members/5183: No such file or directory.
12561 UPDATE 2014-06-17 12:10:04.473 PDT:DETAIL: Could not open file
Andres Freund wrote:
On 2014-06-20 16:50:15 -0400, Alvaro Herrera wrote:
I think most, if not all, the unique indexes declared are part of a
syscache. I don't think we allow those to be null, so in effect those
columns are already not nullable.
Non-unique indexes in indexing.h
Andres Freund and...@2ndquadrant.com writes:
On 2014-06-20 17:29:33 -0400, Tom Lane wrote:
I think Alvaro was complaining that it's alone in lacking this comment:
/* This following index is not used for a cache and is not unique */
But TBH, I don't think those comments are worth much. I'd
Thanks Fujii , that is a bug -- an #ifdef USE_PREFETCH is missing in
heapam.c
(maybe several)
I will fix it in the next patch version.
I also appreciate it is not easy to review the patch.
There are really 4 (or maybe 5) parts :
. async io (librt functions)
. buffer
On 2014-06-20 17:38:16 -0400, Alvaro Herrera wrote:
Jeff Janes wrote:
12538 2014-06-17 12:10:02.925 PDT:LOG: JJ deleting 0xb66b20 5183
12498 UPDATE 2014-06-17 12:10:03.188 PDT:DETAIL: Could not open file
pg_multixact/members/5183: No such file or directory.
12561 UPDATE 2014-06-17
I've had a chance to look at this and here is my review.
On 04/14/2014 01:19 PM, David Rowley wrote:
I've included the updated patch with some regression tests.
The first thing I noticed is there is no documentation, but I don't
think we document such things outside of the release notes, so
Robert,
* Robert Haas (robertmh...@gmail.com) wrote:
On Wed, Jun 18, 2014 at 10:40 AM, Stephen Frost sfr...@snowman.net wrote:
* Robert Haas (robertmh...@gmail.com) wrote:
Technically, there are 4 bits left, and that's what we need for
separate privileges.
I'd really hate to chew them
Stephen Frost sfr...@snowman.net writes:
* Robert Haas (robertmh...@gmail.com) wrote:
[ counting bits in ACLs ]
Wouldn't it be fairly painless to widen AclMode from 32 bits to 64,
and thereby double the number of available bits?
That code was all written before we required platforms to have an
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 06/13/2014 07:29 AM, Tom Lane wrote:
Fujii Masao masao.fu...@gmail.com writes:
On Thu, Jun 12, 2014 at 8:51 PM, Fujii Masao
masao.fu...@gmail.com wrote:
Some users enable log_disconnections in postgresql.conf to
audit all logouts. But since
Tom,
* Tom Lane (t...@sss.pgh.pa.us) wrote:
Stephen Frost sfr...@snowman.net writes:
* Robert Haas (robertmh...@gmail.com) wrote:
[ counting bits in ACLs ]
Wouldn't it be fairly painless to widen AclMode from 32 bits to 64,
and thereby double the number of available bits?
Thanks for
33 matches
Mail list logo