Re: [HACKERS] Per-column collation, the finale

2011-01-28 Thread Noah Misch
Hi Peter, I'm reviewing this patch as part of CommitFest 2011-01. On Fri, Jan 14, 2011 at 11:41:46PM +0200, Peter Eisentraut wrote: > I've been going over this patch with a fine-tooth comb for the last two > weeks, fixed some small bugs, added comments, made initdb a little > friendlier, but no s

Re: [HACKERS] Include WAL in base backup

2011-01-28 Thread Fujii Masao
On Sat, Jan 29, 2011 at 6:02 AM, Heikki Linnakangas wrote: > On 27.01.2011 06:44, Fujii Masao wrote: >> >> +               XLByteToSeg(endptr, endlogid, endlogseg); >> >> +                       /* Have we reached our stop position yet? */ >> +                       if (logid>  endlogid || >> +  

Re: [HACKERS] SSI patch version 14

2011-01-28 Thread Dan Ports
On Thu, Jan 27, 2011 at 09:18:23AM -0800, Jeff Davis wrote: > On Tue, 2011-01-25 at 05:57 -0500, Dan Ports wrote: > > This summary is right on. I would add one additional detail or > > clarification to the last point, which is that rather than checking for > > a cycle, we're checking for a transact

Re: [HACKERS] pg_upgrade fails for non-postgres user

2011-01-28 Thread Bruce Momjian
Magnus Hagander wrote: > I just tried doing pg_upgrade on a database when logged in as user > "mha" rather than "postgres" on my system. And it failed. Even though > the db was initialized with superuser "mha". The reason for this was > that pg_upgrade tried to connect to the database "mha" (hardco

Re: [HACKERS] Add support for logging the current role

2011-01-28 Thread Stephen Frost
Itagaki, * Itagaki Takahiro (itagaki.takah...@gmail.com) wrote: > Discussions > * How about "csvlog_fields" rather than "log_csv_fields"? > Since we have variables with syslog_ prefix, csvlog_ prefix > seems to be better. Sure, not a big deal, to be honest, as long as we can actuall

[HACKERS] TG_DEPTH patch v1

2011-01-28 Thread Kevin Grittner
The people who write my paychecks have insisted on me chunking out some items which are part of our long-term plan besides the one I've been focusing on lately. Most of it isn't of interest to anyone outside Wisconsin Courts, but this piece might be; so I'm posting it and putting onto the first 9.

Re: [HACKERS] mingw format warnings

2011-01-28 Thread Andrew Dunstan
On 01/28/2011 05:34 PM, Tom Lane wrote: Andrew Dunstan writes: On 01/28/2011 03:11 PM, Tom Lane wrote: Are any of them substantive, or are they all about %m ? [andrew@aurelia ]$ grep 'warning.*format' frogmouth-make*.log | sed 's/.*warning:/warning:/' | sort | uniq -c I'd say a lot of tho

Re: [HACKERS] pg_upgrade fails for non-postgres user

2011-01-28 Thread Josh Berkus
> When that was fixed, I realized the psql command to create the > datanbases connect to database "template1" only to immediately switch > to database "postgres", which also seems rather pointless. > > Attach patch makes it connect to the "postgres" database instead of > $USER, and then also chan

Re: [HACKERS] mingw format warnings

2011-01-28 Thread Andrew Dunstan
On 01/28/2011 02:39 PM, Peter Eisentraut wrote: On fre, 2011-01-28 at 14:12 -0500, Andrew Dunstan wrote: We have had format warnings in mingw builds for years, and it doesn't look like we're ever going to do anything about them. Can we just add -Wno-format to the CFLAGS for Mingw and be done w

Re: [HACKERS] mingw format warnings

2011-01-28 Thread Tom Lane
Andrew Dunstan writes: > On 01/28/2011 03:11 PM, Tom Lane wrote: >> Are any of them substantive, or are they all about %m ? > [andrew@aurelia ]$ grep 'warning.*format' frogmouth-make*.log | sed > 's/.*warning:/warning:/' | sort | uniq -c I'd say a lot of those probably need attention. It migh

Re: [HACKERS] mingw format warnings

2011-01-28 Thread Andrew Dunstan
On 01/28/2011 03:11 PM, Tom Lane wrote: Andrew Dunstan writes: We have had format warnings in mingw builds for years, and it doesn't look like we're ever going to do anything about them. Can we just add -Wno-format to the CFLAGS for Mingw and be done with them? Are any of them substantive, o

Re: [HACKERS] arrays as pl/perl input arguments [PATCH]

2011-01-28 Thread Alex Hunsaker
On Thu, Jan 27, 2011 at 03:38, Alexey Klyukin wrote: > Hi, > > On Jan 27, 2011, at 9:31 AM, Alex Hunsaker wrote: > >> Find attached v3 of the patch.  changes include: >> - fix deep recursion due to accidental reversal of check in >> encode_array_literal >> - add proper support for stringifying co

[HACKERS] pg_upgrade fails for non-postgres user

2011-01-28 Thread Magnus Hagander
I just tried doing pg_upgrade on a database when logged in as user "mha" rather than "postgres" on my system. And it failed. Even though the db was initialized with superuser "mha". The reason for this was that pg_upgrade tried to connect to the database "mha" (hardcoded to be the db username), and

Re: [HACKERS] WIP: RangeTypes

2011-01-28 Thread Thom Brown
On 28 January 2011 20:28, Thom Brown wrote: > On 28 January 2011 07:45, Jeff Davis wrote: >> Updated patch. >> >> Changes: >> >>  * Documentation for operators/functions >>  * a comprehensive set of operators and functions >>  * BTree opclass >>  * Hash opclass >>  * built-in range types: >>    -

Re: [HACKERS] ALTER TYPE 3: add facility to identify further no-work cases

2011-01-28 Thread Noah Misch
On Fri, Jan 28, 2011 at 01:52:32PM -0500, Robert Haas wrote: > On Wed, Jan 26, 2011 at 6:06 PM, Tom Lane wrote: > > I'm not sure how important that concern is though, because it's hard to > > see how any such change wouldn't break existing cast implementation > > functions anyway. ?If the API for

Re: [HACKERS] Include WAL in base backup

2011-01-28 Thread Heikki Linnakangas
On 27.01.2011 06:44, Fujii Masao wrote: + XLByteToSeg(endptr, endlogid, endlogseg); + /* Have we reached our stop position yet? */ + if (logid> endlogid || + (logid == endlogid&& logseg>= endlogseg)) +

Re: [HACKERS] [PATCH] Return command tag 'REPLACE X' for CREATE OR REPLACE statements.

2011-01-28 Thread Marti Raudsepp
On Fri, Jan 28, 2011 at 21:04, Robert Haas wrote: > On Sat, Jan 22, 2011 at 9:11 PM, Robert Haas wrote: >> It seems no one wants to put any further effort into this problem.  Bummer. > > Since no one has felt the need to dispute the above statement in the > last 6 days, it seems clear to mark thi

Re: [HACKERS] FOR KEY LOCK foreign keys

2011-01-28 Thread Marti Raudsepp
On Thu, Jan 13, 2011 at 23:58, Alvaro Herrera wrote: > It goes like this: instead of acquiring a shared lock on the involved > tuple, we only acquire a "key lock", that is, something that prevents > the tuple from going away entirely but not from updating fields that are > not covered by any uniqu

Re: [HACKERS] FPI

2011-01-28 Thread Robert Haas
On Fri, Jan 28, 2011 at 3:08 PM, Tom Lane wrote: > Robert Haas writes: >> Any substantive comments, besides the obvious "this is not 9.1 material"? > > Now that I've absorbed a bit more caffeine, let's see if I can think > straight this time. > > General principle you want to assert: any WAL entr

Re: [HACKERS] WIP: RangeTypes

2011-01-28 Thread Thom Brown
On 28 January 2011 07:45, Jeff Davis wrote: > Updated patch. > > Changes: > >  * Documentation for operators/functions >  * a comprehensive set of operators and functions >  * BTree opclass >  * Hash opclass >  * built-in range types: >    - PERIOD (timestamp) >    - PERIODTZ (timestamptz) >    -

Re: [HACKERS] WIP: RangeTypes

2011-01-28 Thread Chris Browne
pg...@j-davis.com (Jeff Davis) writes: > On Fri, 2011-01-28 at 09:17 -0800, David Fetter wrote: >> For consistency, and in order not to continue our atrocious naming >> tradition, I'd like to propose that the above be named timestamprange >> (tsrange for short) and timestamptzrange (tstzrange for

Re: [HACKERS] mingw format warnings

2011-01-28 Thread Tom Lane
Andrew Dunstan writes: > We have had format warnings in mingw builds for years, and it doesn't > look like we're ever going to do anything about them. Can we just add > -Wno-format to the CFLAGS for Mingw and be done with them? Are any of them substantive, or are they all about %m ?

Re: [HACKERS] FPI

2011-01-28 Thread Tom Lane
Robert Haas writes: > Any substantive comments, besides the obvious "this is not 9.1 material"? Now that I've absorbed a bit more caffeine, let's see if I can think straight this time. General principle you want to assert: any WAL entry that merely results in setting a deterministic field to a d

Re: [HACKERS] mingw64

2011-01-28 Thread JonY
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 1/28/2011 23:23, Andrew Dunstan wrote: > > > On 01/28/2011 08:09 AM, JonY wrote: >> -BEGIN PGP SIGNED MESSAGE- >> Hash: SHA1 >> >> On 1/28/2011 18:19, Andrew Dunstan wrote: configure:7071: checking for zic configure:7104: result:

Re: [HACKERS] mingw64

2011-01-28 Thread JonY
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 1/28/2011 18:19, Andrew Dunstan wrote: >> configure:7071: checking for zic >> configure:7104: result: no >> configure:7113: error: >> When cross-compiling, either use the option --with-system-tzdata to use >> existing time-zone data, or set the envi

Re: [HACKERS] mingw format warnings

2011-01-28 Thread Peter Eisentraut
On fre, 2011-01-28 at 14:12 -0500, Andrew Dunstan wrote: > We have had format warnings in mingw builds for years, and it doesn't > look like we're ever going to do anything about them. Can we just add > -Wno-format to the CFLAGS for Mingw and be done with them? I wasn't aware of this issue, but

Re: [HACKERS] Re: patch: fix performance problems with repated decomprimation of varlena values in plpgsql

2011-01-28 Thread Pavel Stehule
2011/1/28 Robert Haas : > On Tue, Jan 25, 2011 at 11:29 AM, Pavel Stehule > wrote: >> variant a) my first patch - detoast on first usage with avoiding to >> useless detoast checking >> variant b) my first patch - detoast on first usage without avoiding to >> useless detoast checking >> >> time fo

[HACKERS] mingw format warnings

2011-01-28 Thread Andrew Dunstan
We have had format warnings in mingw builds for years, and it doesn't look like we're ever going to do anything about them. Can we just add -Wno-format to the CFLAGS for Mingw and be done with them? cheers andrew -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make

Re: [HACKERS] FPI

2011-01-28 Thread Robert Haas
On Fri, Jan 28, 2011 at 1:42 PM, Tom Lane wrote: > Robert Haas writes: >> On Fri, Jan 28, 2011 at 10:13 AM, Tom Lane wrote: >>> Say what?  A heap deletion compacts the page --- it will certainly fail >>> badly on torn-page. > >> What do you mean by "compacts the page"?  I would interpret that to

Re: [HACKERS] Re: patch: fix performance problems with repated decomprimation of varlena values in plpgsql

2011-01-28 Thread Robert Haas
On Tue, Jan 25, 2011 at 11:29 AM, Pavel Stehule wrote: > variant a) my first patch - detoast on first usage with avoiding to > useless detoast checking > variant b) my first patch - detoast on first usage without avoiding to > useless detoast checking > > time for 1 - about 300 ms, a is bout 1.5%

Re: [HACKERS] [PATCH] Return command tag 'REPLACE X' for CREATE OR REPLACE statements.

2011-01-28 Thread Robert Haas
On Sat, Jan 22, 2011 at 9:11 PM, Robert Haas wrote: > It seems no one wants to put any further effort into this problem.  Bummer. Since no one has felt the need to dispute the above statement in the last 6 days, it seems clear to mark this Returned with Feedback, which I have now done. -- Rober

Re: [HACKERS] ALTER TYPE 3: add facility to identify further no-work cases

2011-01-28 Thread Robert Haas
On Wed, Jan 26, 2011 at 6:06 PM, Tom Lane wrote: > I'm not sure how important that concern is though, because it's hard to > see how any such change wouldn't break existing cast implementation > functions anyway.  If the API for length-coercing cast functions > changes, breaking their helper funct

Re: [HACKERS] FPI

2011-01-28 Thread Tom Lane
Robert Haas writes: > On Fri, Jan 28, 2011 at 10:13 AM, Tom Lane wrote: >> Say what?  A heap deletion compacts the page --- it will certainly fail >> badly on torn-page. > What do you mean by "compacts the page"? I would interpret that to > mean "reclaims the space formerly used by the tuple be

Re: [HACKERS] WIP: RangeTypes

2011-01-28 Thread David E. Wheeler
On Jan 28, 2011, at 9:48 AM, Jeff Davis wrote: > On Fri, 2011-01-28 at 09:17 -0800, David Fetter wrote: >> For consistency, and in order not to continue our atrocious naming >> tradition, I'd like to propose that the above be named timestamprange >> (tsrange for short) and timestamptzrange (tstzra

Re: [HACKERS] FPI

2011-01-28 Thread Robert Haas
On Fri, Jan 28, 2011 at 10:13 AM, Tom Lane wrote: > Robert Haas writes: >> I was thinking about full-page writes again tonight.  I'm still >> wondering about the feasibility of getting rid of full-page writes for >> certain operations.  We can do this, I think, in any case where we can >> convinc

Re: [HACKERS] WIP: RangeTypes

2011-01-28 Thread Tom Lane
Jeff Davis writes: > On Fri, 2011-01-28 at 09:17 -0800, David Fetter wrote: >> For consistency, and in order not to continue our atrocious naming >> tradition, I'd like to propose that the above be named timestamprange >> (tsrange for short) and timestamptzrange (tstzrange for short). > No real o

Re: [HACKERS] WIP: RangeTypes

2011-01-28 Thread Jeff Davis
On Fri, 2011-01-28 at 09:17 -0800, David Fetter wrote: > For consistency, and in order not to continue our atrocious naming > tradition, I'd like to propose that the above be named timestamprange > (tsrange for short) and timestamptzrange (tstzrange for short). No real objection, but I'd like to s

Re: [HACKERS] WIP: RangeTypes

2011-01-28 Thread David Fetter
On Thu, Jan 27, 2011 at 11:45:30PM -0800, Jeff Davis wrote: > Updated patch. > > Changes: > > * Documentation for operators/functions > * a comprehensive set of operators and functions > * BTree opclass Yay! > * Hash opclass > * built-in range types: > - PERIOD (timestamp) > -

Re: [HACKERS] SSI patch version 14

2011-01-28 Thread Jeff Davis
On Tue, 2011-01-25 at 15:22 -0600, Kevin Grittner wrote: > Jeff Davis wrote: > > > I think just annotating RWConflict.in/outLink and > > PredTranList.available/activeList with the types of things they > > hold would be a help. > > > > Also, you say something about RWConflict and "when the struc

Re: pg_ctl failover Re: [HACKERS] Latches, signals, and waiting

2011-01-28 Thread Tatsuo Ishii
I did s/failover/promote. Here is the updated patch. >>> >>> I rebased the patch to current git master. >> >> I'm thinking about implementing a function which does a promotion for >> the standby. It will make pgpool lot easier to control the promotion >> since it allow to fire the promotion op

Re: pg_ctl failover Re: [HACKERS] Latches, signals, and waiting

2011-01-28 Thread Tom Lane
Robert Haas writes: > On Fri, Jan 28, 2011 at 3:40 AM, Tatsuo Ishii wrote: Agreed. I submitted the patch before, but I forgot to update it and add it to CF. http://archives.postgresql.org/message-id/aanlktimuhbxbum+zlkaex3adqseimue3xb4ww1qts...@mail.gmail.com >> >> Great! > I hat

Re: [HACKERS] mingw64

2011-01-28 Thread Andrew Dunstan
On 01/28/2011 08:09 AM, JonY wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 1/28/2011 18:19, Andrew Dunstan wrote: configure:7071: checking for zic configure:7104: result: no configure:7113: error: When cross-compiling, either use the option --with-system-tzdata to use existing time-

Re: [HACKERS] FPI

2011-01-28 Thread Tom Lane
Robert Haas writes: > I was thinking about full-page writes again tonight. I'm still > wondering about the feasibility of getting rid of full-page writes for > certain operations. We can do this, I think, in any case where we can > convince ourselves that if the original operation, or a redo of

Re: [HACKERS] make -j2 error in ecpg/pgtypeslib in current GIT

2011-01-28 Thread Boszormenyi Zoltan
2011-01-27 16:24 keltezéssel, Tom Lane írta: > Boszormenyi Zoltan writes: >> I get reproducible "make -j2" compile error in ecpg/pgtypeslib. > FWIW, I can't reproduce that at all, either at -j2 or -j4, on either > Fedora 13 or Fedora 14. > > regards, tom lane FWIW, I run "ma

Re: pg_ctl failover Re: [HACKERS] Latches, signals, and waiting

2011-01-28 Thread Robert Haas
On Fri, Jan 28, 2011 at 3:40 AM, Tatsuo Ishii wrote: >> On Fri, Jan 28, 2011 at 4:57 PM, Magnus Hagander wrote: >>> On Fri, Jan 28, 2011 at 08:44, Tatsuo Ishii wrote: >> I did s/failover/promote. Here is the updated patch. > > I rebased the patch to current git master. I'm

Re: [HACKERS] mingw64

2011-01-28 Thread Andrew Dunstan
On 01/27/2011 10:51 PM, Xiaobo Gu wrote: On Fri, Jan 28, 2011 at 11:44 AM, Andrew Dunstan wrote: > > > On 01/27/2011 10:37 PM, Xiaobo Gu wrote: >> >> --with-system-tzdata=/usr/share/zoneinfo > > Why on earth are you doing this on Windows? That's crazy. Did you run "make > check"? You sh

Re: pg_ctl failover Re: [HACKERS] Latches, signals, and waiting

2011-01-28 Thread Tatsuo Ishii
> On Fri, Jan 28, 2011 at 4:57 PM, Magnus Hagander wrote: >> On Fri, Jan 28, 2011 at 08:44, Tatsuo Ishii wrote: > I did s/failover/promote. Here is the updated patch. I rebased the patch to current git master. >>> >>> I'm thinking about implementing a function which does a promotion

Re: pg_ctl failover Re: [HACKERS] Latches, signals, and waiting

2011-01-28 Thread Fujii Masao
On Fri, Jan 28, 2011 at 4:57 PM, Magnus Hagander wrote: > On Fri, Jan 28, 2011 at 08:44, Tatsuo Ishii wrote: I did s/failover/promote. Here is the updated patch. >>> >>> I rebased the patch to current git master. >> >> I'm thinking about implementing a function which does a promotion for >>

Re: [HACKERS] Transaction-scope advisory locks

2011-01-28 Thread Marko Tiikkaja
On 1/23/2011 4:24 AM, Robert Haas wrote: On Thu, Jan 20, 2011 at 5:22 AM, Marko Tiikkaja wrote: On 2011-01-17 9:28 AM +0200, Itagaki Takahiro wrote: == Coding == I expect documentation will come soon. I'm sorry about this, I have been occupied with other stuff. I'm going to work on this to

Re: pg_ctl failover Re: [HACKERS] Latches, signals, and waiting

2011-01-28 Thread Tatsuo Ishii
> On Fri, Jan 28, 2011 at 08:44, Tatsuo Ishii wrote: I did s/failover/promote. Here is the updated patch. >>> >>> I rebased the patch to current git master. >> >> I'm thinking about implementing a function which does a promotion for >> the standby. It will make pgpool lot easier to control th