Re: [HACKERS] Audit of logout

2014-08-22 Thread Amit Kapila
On Tue, Aug 5, 2014 at 8:04 PM, Fujii Masao wrote: > > Yep, the attached patch introduces PGC_SU_BACKEND and > changes the contexts of log_connections and log_disconnections > to PGC_SU_BACKEND. Review? > 1. ! else if (context != PGC_POSTMASTER && context != PGC_SU_BACKEND && ! context != PGC_SU

Re: [HACKERS] Support for N synchronous standby servers

2014-08-22 Thread Michael Paquier
On Fri, Aug 22, 2014 at 11:42 PM, Michael Paquier wrote: >> >> 2. Logic of deciding the highest priority one seems to be in-correct. >> Assume, s_s_num = 3, s_s_names = 3,4,2,1 >> standby nodes are in order as: 1,2,3,4,5,6,7 >> >> As per the logic in patch, node 4 with prio

Re: [HACKERS] Re: proposal: ignore null fields in not relation type composite type based constructors

2014-08-22 Thread Pavel Stehule
Hello 2014-08-22 12:21 GMT+02:00 Jeevan Chalke : > Hi Pavel, > > You have said that XMLFOREST has something which ignores nulls, what's > that? > Will you please provide an example ? > I was partially wrong - XMLFOREST ignore null always postgres=# select xmlforest(10 as a,20 as b,null as c);

Re: [HACKERS] proposal: rounding up time value less than its unit.

2014-08-22 Thread Tomonari Katsumata
Thank you for the comments. It was a bug in my patch as another developer says. I've not considered about the value 'zero', sorry. I attached new patch. This patch rounds up the value when only it's less than required unit. Like below. (unit: min) 0->0 0s->0 10s->1 70s->1 Although my original co

Re: [HACKERS] Is this a bug?

2014-08-22 Thread Bruce Momjian
On Fri, Aug 22, 2014 at 03:12:47PM -0400, Robert Haas wrote: > On Fri, Aug 22, 2014 at 2:33 PM, Bruce Momjian wrote: > >> Yes, you remember well. I will have to find a different way for > >> pg_upgrade to call a no-op ALTER TABLE, which is fine. > > > > Looking at the ALTER TABLE options, I am go

Re: [HACKERS] [v9.5] Custom Plan API

2014-08-22 Thread Kohei KaiGai
2014-08-23 0:39 GMT+09:00 Robert Haas : > On Thu, Jul 17, 2014 at 3:38 PM, Tom Lane wrote: >> Alvaro Herrera writes: >>> I haven't followed this at all, but I just skimmed over it and noticed >>> the CustomPlanMarkPos thingy; apologies if this has been discussed >>> before. It seems a bit odd to

Re: [HACKERS] 9.5: Better memory accounting, towards memory-bounded HashAgg

2014-08-22 Thread Tomas Vondra
On 20.8.2014 08:11, Jeff Davis wrote: > On Tue, 2014-08-19 at 12:54 +0200, Tomas Vondra wrote: > > It would be easier to resolve the performance concern if I could > reliably get the results Robert is getting. I think I was able to > reproduce the regression with the old patch, but the results were

Re: [HACKERS] [GSoC2014] Patch ALTER TABLE ... SET LOGGED

2014-08-22 Thread Fabrízio de Royes Mello
On Fri, Aug 22, 2014 at 4:45 PM, Alvaro Herrera wrote: > > BTW why is it that index_build() checks the heap's relpersistence flag > rather than the index'? > I'm curious about it too... the code in src/backend/catalog/index.c is: 1975 if (heapRelation->rd_rel->relpersistence == RELPERSISTENC

Re: [HACKERS] WIP Patch for GROUPING SETS phase 1

2014-08-22 Thread Andrew Gierth
> "Tom" == Tom Lane writes: Tom> I'm not convinced of that; I think some creative hackery in the Tom> grammar might be able to deal with this. Making GROUP BY CUBE(a,b) parse as grouping sets rather than as a function turned out to be the easy part: give CUBE a lower precedence than '(' (e

Re: [HACKERS] SKIP LOCKED DATA (work in progress)

2014-08-22 Thread Alvaro Herrera
heap_lock_tuple() has the following comment on top: * In the failure cases, the routine fills *hufd with the tuple's t_ctid, * t_xmax (resolving a possible MultiXact, if necessary), and t_cmax * (the last only for HeapTupleSelfUpdated, since we * cannot obtain cmax from a combocid generated by

Re: [HACKERS] WIP Patch for GROUPING SETS phase 1

2014-08-22 Thread Stephen Frost
* Greg Stark (st...@mit.edu) wrote: > On Fri, Aug 22, 2014 at 10:37 PM, Stephen Frost wrote: > > Agreed- and how many of those have *every extension available* loaded... > > Actually that was also in the talk.a few slides later. 0.7% So, 0.3% install cube w/o installing *every* extension..? Tha

Re: [HACKERS] WIP Patch for GROUPING SETS phase 1

2014-08-22 Thread Greg Stark
On Fri, Aug 22, 2014 at 10:37 PM, Stephen Frost wrote: > Agreed- and how many of those have *every extension available* loaded... Actually that was also in the talk.a few slides later. 0.7% -- greg -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your

Re: [HACKERS] WIP Patch for GROUPING SETS phase 1

2014-08-22 Thread Stephen Frost
* Andrew Dunstan (and...@dunslane.net) wrote: > > On 08/22/2014 02:42 PM, Greg Stark wrote: > >On Fri, Aug 22, 2014 at 7:02 PM, Tom Lane wrote: > >>So the proposal you are pushing is going > >>to result in seriously teeing off some fraction of our userbase; > >>and the argument why that would be

Re: [HACKERS] WIP Patch for GROUPING SETS phase 1

2014-08-22 Thread Merlin Moncure
On Fri, Aug 22, 2014 at 1:52 PM, Robert Haas wrote: > On Fri, Aug 22, 2014 at 2:02 PM, Tom Lane wrote: > https://www.youtube.com/watch?v=MT2gzzbyWpw > > At around 8 minutes, he shows utilization statistics for cube at > around 1% across their install base. That's higher than I would have > guess

Re: [HACKERS] WIP Patch for GROUPING SETS phase 1

2014-08-22 Thread Andrew Dunstan
On 08/22/2014 02:42 PM, Greg Stark wrote: On Fri, Aug 22, 2014 at 7:02 PM, Tom Lane wrote: So the proposal you are pushing is going to result in seriously teeing off some fraction of our userbase; and the argument why that would be acceptable seems to boil down to "I think there are few enough

Re: [HACKERS] [GSoC2014] Patch ALTER TABLE ... SET LOGGED

2014-08-22 Thread Tom Lane
Alvaro Herrera writes: > Robert Haas wrote: >> 1. Loggedness is not a word. I think that "persistence" or >> "relpersistence" would be better. > You want me to change that to chgPersistence and so on? No prob, just > LMK. +1 for s/loggedness/persistence/ -- I agree with Robert that it's a bit

Re: [HACKERS] [GSoC2014] Patch ALTER TABLE ... SET LOGGED

2014-08-22 Thread Tom Lane
Robert Haas writes: > 2. The patch seems to think that it can sometimes be safe to change > the relpersistence of an existing relation. Unless you can be sure > that no buffers can possibly be present in shared_buffers and nobody > will use an existing relcache entry to read a new one in, it's no

Re: [HACKERS] [GSoC2014] Patch ALTER TABLE ... SET LOGGED

2014-08-22 Thread Alvaro Herrera
Fabrízio de Royes Mello wrote: > On Fri, Aug 22, 2014 at 4:45 PM, Alvaro Herrera > wrote: > > I pointed out, in the email just before pushing the patch, that perhaps > > we should pass down the new relpersistence flag into finish_heap_swap, > > and from there we could pass it down to reindex_inde

Re: [HACKERS] [GSoC2014] Patch ALTER TABLE ... SET LOGGED

2014-08-22 Thread Fabrízio de Royes Mello
On Fri, Aug 22, 2014 at 4:45 PM, Alvaro Herrera wrote: > > > 2. The patch seems to think that it can sometimes be safe to change > > the relpersistence of an existing relation. Unless you can be sure > > that no buffers can possibly be present in shared_buffers and nobody > > will use an existing

Re: [HACKERS] [GSoC2014] Patch ALTER TABLE ... SET LOGGED

2014-08-22 Thread Alvaro Herrera
Fabrízio de Royes Mello wrote: > On Fri, Aug 22, 2014 at 4:22 PM, Robert Haas wrote: > > 2. The patch seems to think that it can sometimes be safe to change > > the relpersistence of an existing relation. Unless you can be sure > > that no buffers can possibly be present in shared_buffers and no

Re: [HACKERS] [GSoC2014] Patch ALTER TABLE ... SET LOGGED

2014-08-22 Thread Alvaro Herrera
Robert Haas wrote: > Hmm. I confess to not having paid enough attention to this, Sorry about that. I guess I should somehow flag threads "I'm planning to commit this" so that other people can review stuff carefully. > but: > > 1. Loggedness is not a word. I think that "persistence" or > "relp

Re: [HACKERS] [GSoC2014] Patch ALTER TABLE ... SET LOGGED

2014-08-22 Thread Fabrízio de Royes Mello
On Fri, Aug 22, 2014 at 4:22 PM, Robert Haas wrote: > > On Fri, Aug 22, 2014 at 2:32 PM, Alvaro Herrera > wrote: > > Fabrízio de Royes Mello wrote: > >> Em sexta-feira, 22 de agosto de 2014, Alvaro Herrera < > >> alvhe...@2ndquadrant.com> escreveu: > >> > >> > Fabrízio de Royes Mello wrote: > >>

Re: [HACKERS] SKIP LOCKED DATA (work in progress)

2014-08-22 Thread Alvaro Herrera
One thing I just noticed is that we uselessly set an error context callback when "waiting" in ConditionalMultiXactIdWait, which is pretty useless (because we don't actually wait there at all) -- we don't set one in ConditionalXactLockTableWait, which makes sense, but for some reason I failed to rea

Re: [HACKERS] 9.5: Better memory accounting, towards memory-bounded HashAgg

2014-08-22 Thread Robert Haas
On Fri, Aug 22, 2014 at 2:13 PM, Tomas Vondra wrote: > I don't think we really need to abandon the 'tracked' flag (or that we > should). I think it was useful, and removing it might be one of the > reasons why Robert now sees worse impact than before. The version that introduced that flag had the

Re: [HACKERS] [GSoC2014] Patch ALTER TABLE ... SET LOGGED

2014-08-22 Thread Robert Haas
On Fri, Aug 22, 2014 at 2:32 PM, Alvaro Herrera wrote: > Fabrízio de Royes Mello wrote: >> Em sexta-feira, 22 de agosto de 2014, Alvaro Herrera < >> alvhe...@2ndquadrant.com> escreveu: >> >> > Fabrízio de Royes Mello wrote: >> > >> > > I forgot to mention... I did again a lot of tests using differ

Re: [HACKERS] [PATCH] Incremental backup: add backup profile to base backup

2014-08-22 Thread Bruce Momjian
On Fri, Aug 22, 2014 at 12:00:19PM -0400, Robert Haas wrote: > On Wed, Aug 20, 2014 at 7:33 PM, Claudio Freire > wrote: > > On Wed, Aug 20, 2014 at 8:24 PM, Bruce Momjian wrote: > >> On Mon, Aug 18, 2014 at 04:05:07PM +0300, Heikki Linnakangas wrote: > >>> But more to the point, I thought the co

Re: [HACKERS] Is this a bug?

2014-08-22 Thread Robert Haas
On Fri, Aug 22, 2014 at 2:33 PM, Bruce Momjian wrote: >> Yes, you remember well. I will have to find a different way for >> pg_upgrade to call a no-op ALTER TABLE, which is fine. > > Looking at the ALTER TABLE options, I am going to put this check in a > !IsBinaryUpgrade block so pg_upgrade can s

Re: [HACKERS] Is this a bug?

2014-08-22 Thread Andres Freund
On August 22, 2014 8:33:57 PM CEST, Bruce Momjian wrote: >On Fri, Aug 22, 2014 at 12:53:30PM -0400, Bruce Momjian wrote: >> On Fri, Aug 22, 2014 at 10:27:02AM -0400, Robert Haas wrote: >> > On Thu, Aug 21, 2014 at 7:17 PM, Bruce Momjian >wrote: >> > > On Tue, Mar 18, 2014 at 09:11:46AM -0400, Rob

Re: [HACKERS] WIP Patch for GROUPING SETS phase 1

2014-08-22 Thread Robert Haas
On Fri, Aug 22, 2014 at 2:02 PM, Tom Lane wrote: > Robert Haas writes: >> On Thu, Aug 21, 2014 at 2:13 PM, Tom Lane wrote: >>> Well, if there are any extant applications that use that exact phrasing, >>> they're going to be broken in any case. That does not mean that we have >>> to break every

Re: [HACKERS] B-Tree support function number 3 (strxfrm() optimization)

2014-08-22 Thread Peter Geoghegan
On Fri, Aug 22, 2014 at 7:19 AM, Robert Haas wrote: > Patch 0002 no longer applies; please rebase. I attach rebased patch. Note that there is currently a bug in the master branch: + if (len2 >= tss->buflen2) + { + pfree(tss->buf2); + tss->buflen1 = Max(len2 + 1, Min(tss->buflen2

Re: [HACKERS] WIP Patch for GROUPING SETS phase 1

2014-08-22 Thread Greg Stark
On Fri, Aug 22, 2014 at 7:02 PM, Tom Lane wrote: > So the proposal you are pushing is going > to result in seriously teeing off some fraction of our userbase; > and the argument why that would be acceptable seems to boil down to > "I think there are few enough of them that we don't have to care" >

Re: [HACKERS] [GSoC2014] Patch ALTER TABLE ... SET LOGGED

2014-08-22 Thread Fabrízio de Royes Mello
On Fri, Aug 22, 2014 at 3:32 PM, Alvaro Herrera wrote: > > Fabrízio de Royes Mello wrote: > > Em sexta-feira, 22 de agosto de 2014, Alvaro Herrera < > > alvhe...@2ndquadrant.com> escreveu: > > > > > Fabrízio de Royes Mello wrote: > > > > > > > I forgot to mention... I did again a lot of tests usin

Re: [HACKERS] Is this a bug?

2014-08-22 Thread Bruce Momjian
On Fri, Aug 22, 2014 at 12:53:30PM -0400, Bruce Momjian wrote: > On Fri, Aug 22, 2014 at 10:27:02AM -0400, Robert Haas wrote: > > On Thu, Aug 21, 2014 at 7:17 PM, Bruce Momjian wrote: > > > On Tue, Mar 18, 2014 at 09:11:46AM -0400, Robert Haas wrote: > > >> On Mon, Mar 17, 2014 at 10:27 PM, Michae

Re: [HACKERS] [GSoC2014] Patch ALTER TABLE ... SET LOGGED

2014-08-22 Thread Alvaro Herrera
Fabrízio de Royes Mello wrote: > Em sexta-feira, 22 de agosto de 2014, Alvaro Herrera < > alvhe...@2ndquadrant.com> escreveu: > > > Fabrízio de Royes Mello wrote: > > > > > I forgot to mention... I did again a lot of tests using different > > > replication scenarios to make sure all is ok: > > > -

Re: [HACKERS] 9.5: Better memory accounting, towards memory-bounded HashAgg

2014-08-22 Thread Tomas Vondra
On 20.8.2014 08:11, Jeff Davis wrote: > On Tue, 2014-08-19 at 12:54 +0200, Tomas Vondra wrote: >> The use-case for this is tracking a chosen subtree of contexts - e.g. >> aggcontext and below, so I'd expect the tracked subtrees to be relatively >> shallow. Am I right? > > Right. > >> My fear is t

Re: [HACKERS] WIP Patch for GROUPING SETS phase 1

2014-08-22 Thread Tom Lane
Robert Haas writes: > On Thu, Aug 21, 2014 at 2:13 PM, Tom Lane wrote: >> Well, if there are any extant applications that use that exact phrasing, >> they're going to be broken in any case. That does not mean that we have >> to break every other appearance of "cube". I think that special-casing

Re: [HACKERS] WIP Patch for GROUPING SETS phase 1

2014-08-22 Thread Robert Haas
On Thu, Aug 21, 2014 at 2:13 PM, Tom Lane wrote: > Andrew Gierth writes: >> "Tom" == Tom Lane writes: >> Tom> I wonder if you've tried hard enough to avoid reserving the keyword. > >> GROUP BY cube(a,b) is currently legal syntax and means something completely >> incompatible to what the spec r

Re: [HACKERS] Is this a bug?

2014-08-22 Thread Bruce Momjian
On Fri, Aug 22, 2014 at 10:27:02AM -0400, Robert Haas wrote: > On Thu, Aug 21, 2014 at 7:17 PM, Bruce Momjian wrote: > > On Tue, Mar 18, 2014 at 09:11:46AM -0400, Robert Haas wrote: > >> On Mon, Mar 17, 2014 at 10:27 PM, Michael Paquier > >> wrote: > >> > On Tue, Mar 18, 2014 at 10:24 AM, Fabrízi

Re: [HACKERS] 9.5: Better memory accounting, towards memory-bounded HashAgg

2014-08-22 Thread Robert Haas
On Wed, Aug 20, 2014 at 2:11 AM, Jeff Davis wrote: > I attached a patch that uses two uint32 fields so that it doesn't > increase the size of MemoryContextData, and it tracks memory usage for > all contexts. I was unable to detect any performance regression vs. > master, but on my machine the resu

Re: [HACKERS] change alter user to be a true alias for alter role

2014-08-22 Thread Jov
I make the v2 of the patch,use Tom's advice. But I can't make ROLE and USER in the keyword list,it is hard to solve the conflict,or rewrite many gram rules. the problem is : role_or_user : ROLE | USER; xx_keyword:...| ROLE|...|USER..; this two rules produce conflict.So in v2,I remove ROLE and US

Re: [HACKERS] [v9.5] Custom Plan API

2014-08-22 Thread Tom Lane
Robert Haas writes: > I think the threshold question for this incarnation of the patch is > whether we're happy with new DDL (viz, CREATE CUSTOM PLAN PROVIDER) as > a way of installing new plan providers into the database. I tend to agree with your conclusion that that's a whole lot of infrastruc

Re: [HACKERS] [PATCH] Incremental backup: add backup profile to base backup

2014-08-22 Thread Robert Haas
On Wed, Aug 20, 2014 at 7:33 PM, Claudio Freire wrote: > On Wed, Aug 20, 2014 at 8:24 PM, Bruce Momjian wrote: >> On Mon, Aug 18, 2014 at 04:05:07PM +0300, Heikki Linnakangas wrote: >>> But more to the point, I thought the consensus was to use the >>> highest LSN of all the blocks in the file, no

Re: [HACKERS] [PATCH] Incremental backup: add backup profile to base backup

2014-08-22 Thread Robert Haas
On Mon, Aug 18, 2014 at 4:55 PM, Heikki Linnakangas wrote: > You're not thinking evil enough ;-). Let's say that you have a table that > stores bank transfers. You can do a bank transfer to pay a merchant, get the > goods delivered to you, and then a second transfer to yourself with a > specially

Re: [HACKERS] [v9.5] Custom Plan API

2014-08-22 Thread Robert Haas
On Thu, Jul 17, 2014 at 3:38 PM, Tom Lane wrote: > Alvaro Herrera writes: >> I haven't followed this at all, but I just skimmed over it and noticed >> the CustomPlanMarkPos thingy; apologies if this has been discussed >> before. It seems a bit odd to me; why isn't it sufficient to have a >> bool

Re: [HACKERS] WIP Patch for GROUPING SETS phase 1

2014-08-22 Thread Andrew Gierth
> "Alvaro" == Alvaro Herrera writes: >> (This of course means that if someone has a cube() function call >> in a group by clause of a view, then upgrading will change the >> meaning of the view and possibly fail to create it; there seems to >> be no fix for this, not even using the latest

Re: [HACKERS] Proposal to add a QNX 6.5 port to PostgreSQL

2014-08-22 Thread Andres Freund
On 2014-08-22 10:41:55 -0400, Alvaro Herrera wrote: > Andres Freund wrote: > > Hi, > > > > On 2014-08-21 15:25:44 +, Baker, Keith [OCDUS Non-J&J] wrote: > > > About SA_RESTART: > > > > > > I would like to offer you a different perspective which may alter your > > > cu

Re: [HACKERS] potential bug in psql

2014-08-22 Thread Szymon Guz
On 22 August 2014 17:06, Tom Lane wrote: > Szymon Guz writes: > > when I run `\s` in psql, I get the nice list of queries with an error at > > the end: > > > "\s > > could not save history to file "/dev/tty": No such file or directory" > > Well, that's interesting ... what version of which readl

Re: [HACKERS] potential bug in psql

2014-08-22 Thread Alvaro Herrera
Tom Lane wrote: > TBH, though, \s doesn't seem to me to be anywhere near worth the amount > of work we've already put into it, let alone a major new implementation > effort. Who's for just removing the command altogether? I use it every once in a while. Is there a replacement? There are severa

Re: [HACKERS] Proposal to add a QNX 6.5 port to PostgreSQL

2014-08-22 Thread
I am reaching out to our QNX support contacts today, I will let you know how they respond. Keith Baker > -Original Message- > From: Alvaro Herrera [mailto:alvhe...@2ndquadrant.com] > Sent: Friday, August 22, 2014 10:42 AM > To: Andres Freund > Cc: Baker, Keith [OCDUS Non-J&J]; Robert Haa

Re: [HACKERS] potential bug in psql

2014-08-22 Thread Tom Lane
Szymon Guz writes: > when I run `\s` in psql, I get the nice list of queries with an error at > the end: > "\s > could not save history to file "/dev/tty": No such file or directory" Well, that's interesting ... what version of which readline library are you using? Presumably what is happening

Re: [HACKERS] delta relations in AFTER triggers

2014-08-22 Thread Kevin Grittner
Amit Khandekar wrote: > On 15 August 2014 04:04, Kevin Grittner wrote: >> The identifiers in the trigger function are not resolved to >> particular objects until there is a request to fire the trigger. > Ah ok, you are talking about changes specific to the PL language > handlers. Yes, I agree th

Re: [HACKERS] Support for N synchronous standby servers

2014-08-22 Thread Michael Paquier
On Fri, Aug 22, 2014 at 7:14 PM, Rajeev rastogi wrote: > I have just started looking into this patch. > Please find below my first level of observation from the patch: > Thanks! Updated patch attached. > 1. Allocation of memory for sync_nodes in function > SyncRepGetSynchronousNodes should be

Re: [HACKERS] Proposal to add a QNX 6.5 port to PostgreSQL

2014-08-22 Thread Alvaro Herrera
Andres Freund wrote: > Hi, > > On 2014-08-21 15:25:44 +, Baker, Keith [OCDUS Non-J&J] wrote: > > About SA_RESTART: > > > > I would like to offer you a different perspective which may alter your > > current opinion. > > I believe the port.h QNX macro replacement for SA

Re: [HACKERS] WIP Patch for GROUPING SETS phase 1

2014-08-22 Thread Alvaro Herrera
Andrew Gierth wrote: > (This of course means that if someone has a cube() function call in > a group by clause of a view, then upgrading will change the meaning > of the view and possibly fail to create it; there seems to be no fix > for this, not even using the latest pg_dump, since pg_dump relie

Re: [HACKERS] After switching primary server while using replication slot.

2014-08-22 Thread Andres Freund
Hi, On 2014-08-20 13:14:30 -0400, Robert Haas wrote: > On Tue, Aug 19, 2014 at 6:25 AM, Fujii Masao wrote: > > On Mon, Aug 18, 2014 at 11:16 PM, Sawada Masahiko > > wrote: > >> Hi all, > >> After switching primary serer while using repliaction slot, the > >> standby server will not able to conn

Re: [HACKERS] Proposal to add a QNX 6.5 port to PostgreSQL

2014-08-22 Thread Andres Freund
Hi, On 2014-08-21 15:25:44 +, Baker, Keith [OCDUS Non-J&J] wrote: > About SA_RESTART: > > I would like to offer you a different perspective which may alter your > current opinion. > I believe the port.h QNX macro replacement for SA_RESTART is still a > reasonable sol

Re: [HACKERS] Is this a bug?

2014-08-22 Thread Robert Haas
On Thu, Aug 21, 2014 at 7:17 PM, Bruce Momjian wrote: > On Tue, Mar 18, 2014 at 09:11:46AM -0400, Robert Haas wrote: >> On Mon, Mar 17, 2014 at 10:27 PM, Michael Paquier >> wrote: >> > On Tue, Mar 18, 2014 at 10:24 AM, Fabrízio de Royes Mello >> > wrote: >> >> >> >> On Thu, Mar 13, 2014 at 10:22

Re: [HACKERS] B-Tree support function number 3 (strxfrm() optimization)

2014-08-22 Thread Robert Haas
On Thu, Aug 14, 2014 at 1:24 PM, Peter Geoghegan wrote: > On Thu, Aug 14, 2014 at 9:13 AM, Robert Haas wrote: >> Committed that way. As the patch is by and large the same as what I >> submitted for this originally, I credited myself as first author and >> you as second author. I hope that seems

Re: [HACKERS] pg_upgrade: allow multiple -o/-O options

2014-08-22 Thread Bruce Momjian
On Fri, Aug 22, 2014 at 10:52:12AM +0200, Pavel Raiskup wrote: > On Thursday 21 of August 2014 18:26:37 Bruce Momjian wrote: > > On Tue, Mar 4, 2014 at 04:52:56PM +0100, Pavel Raiskup wrote: > > > RFE: Consider that you want to run pg_upgrade via some script with some > > > default '-o' option.

Re: [HACKERS] Proposal to add a QNX 6.5 port to PostgreSQL

2014-08-22 Thread Noah Misch
On Fri, Aug 22, 2014 at 09:34:42AM +0200, Andres Freund wrote: > On 2014-08-22 01:36:37 -0400, Noah Misch wrote: > > On Thu, Aug 21, 2014 at 01:33:38AM +0200, Andres Freund wrote: > > > On 2014-07-25 18:29:53 -0400, Tom Lane wrote: > > > > > * QNX lacks sigaction SA_RESTART: I modified > >

Re: [HACKERS] WIP Patch for GROUPING SETS phase 1

2014-08-22 Thread Stephen Frost
* Andrew Gierth (and...@tao11.riddles.org.uk) wrote: > Having now spent some more time looking, I believe there is a solution > which makes it unreserved which does not require any significant pain > in the code. I'm not entirely convinced that this is the right > approach in the long term, but it

Re: [HACKERS] add line number as prompt option to psql

2014-08-22 Thread Jeevan Chalke
> I would like to ignore this as UINTMAX lines are too much for a input > > buffer to hold. It is almost NIL chances to hit this. > > Yeah, most likely you will run out of memory before reaching that point, > or out of patience. > > Yep. BTW, I have marked this as "waiting for committer". Thanks

Re: [HACKERS] option -T in pg_basebackup doesn't work on windows

2014-08-22 Thread Heikki Linnakangas
On 08/22/2014 11:35 AM, Amit Kapila wrote: On Fri, Aug 22, 2014 at 1:00 PM, Heikki Linnakangas wrote: On 08/22/2014 07:08 AM, Amit Kapila wrote: Today morning, I realised that there is one problem with the patch I sent yesterday and the problem is that incase user has not given -T option, it w

Re: [HACKERS] Re: proposal: ignore null fields in not relation type composite type based constructors

2014-08-22 Thread Jeevan Chalke
Hi Pavel, You have said that XMLFOREST has something which ignores nulls, what's that? Will you please provide an example ? I am NOT sure, but here you are trying to omit entire field from the output when its value is NULL. But that will add an extra efforts at other end which is using output of

Re: [HACKERS] Support for N synchronous standby servers

2014-08-22 Thread Rajeev rastogi
On 09 August 2014 11:33, Michael Paquier Wrote: > Please find attached a patch to add support of synchronous replication > for multiple standby servers. This is controlled by the addition of a > new GUC parameter called synchronous_standby_num, that makes server > wait for transaction commit on th

Re: [HACKERS] inherit support for foreign tables

2014-08-22 Thread Etsuro Fujita
(2014/08/22 12:58), Alvaro Herrera wrote: Noah Misch wrote: I'm anticipating a bug report along these lines: I saw poor estimates involving a child foreign table, so I ran "ANALYZE VERBOSE", which reported 'INFO: analyzing "public.parent" inheritance tree'. Estimates remained poor,

Re: [HACKERS] Are postgresql-9.4 binaries available on Windows XP?

2014-08-22 Thread Dave Page
Hi On Fri, Aug 22, 2014 at 4:45 AM, Hiroshi Inoue wrote: > Hi Dave and Andrew, > > I recently noticed the thread > [BUGS] BUG #11039: installation fails when trying to install C++ > redistributable . > > Unfortunately I have no XP machine at hand and can't test the > installer by myself. > > Lo

Re: [HACKERS] pg_upgrade: allow multiple -o/-O options

2014-08-22 Thread Pavel Raiskup
On Thursday 21 of August 2014 18:26:37 Bruce Momjian wrote: > On Tue, Mar 4, 2014 at 04:52:56PM +0100, Pavel Raiskup wrote: > > RFE: Consider that you want to run pg_upgrade via some script with some > > default '-o' option. But then you also want to give the script's user a > > chance to specif

Re: [HACKERS] option -T in pg_basebackup doesn't work on windows

2014-08-22 Thread Amit Kapila
On Fri, Aug 22, 2014 at 1:00 PM, Heikki Linnakangas wrote: > On 08/22/2014 07:08 AM, Amit Kapila wrote: >> Today morning, I realised that there is one problem with the >> patch I sent yesterday and the problem is that incase user >> has not given -T option, it will not be able to create the symlin

Re: [HACKERS] Removing dependency to wsock32.lib when compiling code on WIndows

2014-08-22 Thread Michael Paquier
On Fri, Aug 15, 2014 at 8:00 PM, Noah Misch wrote: > > On Fri, Aug 15, 2014 at 12:49:36AM -0700, Michael Paquier wrote: > > Btw, how do you determine if MSVC is using HAVE_GETADDRINFO? Is it > > decided by the inclusion of getaddrinfo.c in @pgportfiles of > > Mkvdbuild.pm? > > src/include/pg_confi

Re: [HACKERS] Proposal to add a QNX 6.5 port to PostgreSQL

2014-08-22 Thread Andres Freund
On 2014-08-22 01:36:37 -0400, Noah Misch wrote: > On Thu, Aug 21, 2014 at 01:33:38AM +0200, Andres Freund wrote: > > On 2014-07-25 18:29:53 -0400, Tom Lane wrote: > > > > * QNX lacks sigaction SA_RESTART: I modified > > > > "src/include/port.h" to define macros to retry system calls upon E

[HACKERS] potential bug in psql

2014-08-22 Thread Szymon Guz
Hi, when I run `\s` in psql, I get the nice list of queries with an error at the end: "\s could not save history to file "/dev/tty": No such file or directory" Newest ubuntu from trunk PostgreSQL 9.5devel on x86_64-unknown-linux-gnu, compiled by Ubuntu clang version 3.4-1ubuntu3 (tags/RELEASE_

Re: [HACKERS] option -T in pg_basebackup doesn't work on windows

2014-08-22 Thread Heikki Linnakangas
On 08/22/2014 07:08 AM, Amit Kapila wrote: On Thu, Aug 21, 2014 at 3:44 PM, Amit Kapila wrote: On Tue, Aug 19, 2014 at 9:51 AM, Amit Kapila wrote: On Mon, Aug 18, 2014 at 7:50 PM, Heikki Linnakangas < hlinnakan...@vmware.com> wrote: Wouldn't it make a lot more sense to create it correctly

Re: [HACKERS] WIP Patch for GROUPING SETS phase 1

2014-08-22 Thread Andrew Gierth
> "Tom" == Tom Lane writes: Tom> Perhaps so. I would really prefer not to have to get into Tom> estimating how many people will be inconvenienced how badly. Tom> It's clear to me that not a lot of sweat has been put into Tom> seeing if we can avoid reserving the keyword, and I think we

Re: [HACKERS] pgcrypto: PGP signatures

2014-08-22 Thread Marko Tiikkaja
On 8/22/14, 2:57 AM, Thomas Munro wrote: I took a quick look at your patch at http://www.postgresql.org/message-id/53edbcf0.9070...@joh.to (sorry I didn't reply directly as I didn't have the message). It applies cleanly, builds, and the tests pass. I will hopefully have more to say after I've p