[HACKERS] Department of Redundancy Department: makeNode(FuncCall) division

2013-02-09 Thread David Fetter
Folks, Per suggestions and lots of help from Andrew Gierth, please find attached a patch to clean up the call sites for FuncCall nodes, which I'd like to expand centrally rather than in each of the 37 (or 38, but I only redid 37) places where it's called. The remaining one is in src/backend/nodes

Re: [HACKERS] Identity projection

2013-02-09 Thread Amit kapila
On Saturday, February 09, 2013 9:03 AM Tom Lane wrote: Amit kapila writes: >>> if (!is_projection_capable_plan(result_plan) && >>> compare_tlist_exprs(sub_tlist, result_plan->targetlist) ) >> Sorry, the check I suggested in last mail should be as below: >> if (!is_projection_capable_plan(result

Re: [HACKERS] Fwd: Successful post to pgsql-hackers

2013-02-09 Thread Alvaro Herrera
Magnus Hagander escribió: > On Sat, Feb 9, 2013 at 4:31 PM, Peter Geoghegan > wrote: > > On 9 February 2013 15:24, Magnus Hagander wrote: > >> It's in your personal majordomo settings. I don't think it's related > >> to that at all, but it's a flag you set on your subscription, so > >> perhaps yo

[HACKERS] Further info about picksplit-doesn't-support-secondary-split whining

2013-02-09 Thread Tom Lane
I've spent the afternoon reading and documenting gistsplit.c, and finally feel like I understand what's going on in there, particularly with respect to the secondary-split logic of log-spam infamy. I'll be committing the results of that in a bit, but meanwhile: what I now realize is that the "fall

Re: [HACKERS] Fwd: Successful post to pgsql-hackers

2013-02-09 Thread Euler Taveira
On 09-02-2013 13:45, Gurjeet Singh wrote: > BTW, I hope I understand what selfcopy is: send a copy to yourself. Why would > that be turned on by default? > If you want to reply to yourself... -- Euler Taveira de Oliveira - Timbira http://www.timbira.com.br/ PostgreSQL: Consultoria,

Re: PATCH: Split stats file per database WAS: [HACKERS] autovacuum stress-testing our system

2013-02-09 Thread Jeff Janes
On Tue, Feb 5, 2013 at 2:31 PM, Tomas Vondra wrote: > >> We do not already have this. There is no relevant spec. I can't see >> how this could need pg_dump support (but what about pg_upgrade?) > > pg_dump - no > > pg_upgrage - IMHO it should create the pg_stat directory. I don't think > it could

Re: [HACKERS] [JDBC] JPA + enum == Exception

2013-02-09 Thread Vitalii Tymchyshyn
> This is not entirely unrelated to the discussions about allowing > broader use of automatic casting server-side. It seems to me that > on one side of the argument is the idea that strict typing reduces > bugs and doesn't lead to problems with ambiguity, especially as > things change; and on the

Re: [HACKERS] Re: proposal: a width specification for s specifier (format function), fix behave when positional and ordered placeholders are used

2013-02-09 Thread Pavel Stehule
2013/2/9 Dean Rasheed : >> 2013/1/31 Pavel Stehule : >>> I am sending rewritten code > > Nice. I think this will be very useful, and it looks like it now > supports everything that printf() does for %s format specifiers, and > it's good that %I and %L behave the same. Also the code is looking > cle

Re: [HACKERS] pgcrypto seeding problem when ssl=on

2013-02-09 Thread Noah Misch
On Sun, Jan 13, 2013 at 07:01:07PM -0500, Noah Misch wrote: > On Sun, Jan 13, 2013 at 05:46:12PM -0500, Tom Lane wrote: > > Marko Kreen writes: > > > On Fri, Dec 21, 2012 at 10:27 PM, Noah Misch wrote: > > >> How about instead calling RAND_cleanup() after each backend fork? > > > > > Attached is

Re: [HACKERS] Patch: log level change in gistsplit.c

2013-02-09 Thread Tom Lane
Josh Hansen writes: > We've been seeing these same messages flying by our logs for years: > 2012-12-13 22:28:07 GMT LOG: picksplit method for column 2 of index > "xxx_idx" doesn't support secondary split > I see from the linked message that Tom Lane made the recommendation of > ch

Re: [HACKERS] [Review] Add SPI_gettypmod() to return a field's typemod from a TupleDesc

2013-02-09 Thread Mark Wong
n and replacing the couple of XXX comments. I'll add it to the next commitfest. Regards, Mark add_spigettypmod-20130209.diff Description: Binary data -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers

Re: [HACKERS] Fwd: Successful post to pgsql-hackers

2013-02-09 Thread Gurjeet Singh
I tried the lists page and disabled the 'selfcopy' since the 'ackpot' was kinda hidden under the mailing list names. I have updated the settings again to disable ackpost, and enable the selfcopy. This email will be test if that worked. BTW, I hope I understand what selfcopy is: send a copy to you

Re: [HACKERS] Fwd: Successful post to pgsql-hackers

2013-02-09 Thread Magnus Hagander
On Sat, Feb 9, 2013 at 4:31 PM, Peter Geoghegan wrote: > On 9 February 2013 15:24, Magnus Hagander wrote: >> It's in your personal majordomo settings. I don't think it's related >> to that at all, but it's a flag you set on your subscription, so >> perhaps you set it by mistake. > > Then I must h

Re: [HACKERS] Fwd: Successful post to pgsql-hackers

2013-02-09 Thread Peter Geoghegan
On 9 February 2013 15:24, Magnus Hagander wrote: > It's in your personal majordomo settings. I don't think it's related > to that at all, but it's a flag you set on your subscription, so > perhaps you set it by mistake. Then I must have set it by mistake too, when I recently changed e-mail addres

Re: [HACKERS] Fwd: Successful post to pgsql-hackers

2013-02-09 Thread Magnus Hagander
It's in your personal majordomo settings. I don't think it's related to that at all, but it's a flag you set on your subscription, so perhaps you set it by mistake. You shold be able to set it from https://mail.postgresql.org. The setting you're looking for is "ackpost", and you'll want to turn it

[HACKERS] Fwd: Successful post to pgsql-hackers

2013-02-09 Thread Gurjeet Singh
Recently I have started getting these confirmations for every email I send to the mailing lists. I think it's related to the fact that I recently switched to using a new email address. How can I turn these notifications off? -- Forwarded message -- From: Date: Sat, Feb 9, 2013 at

Re: [HACKERS] pg_prewarm

2013-02-09 Thread Gurjeet Singh
On Sun, Jun 24, 2012 at 1:36 PM, Cédric Villemain wrote: > Le samedi 23 juin 2012 02:47:15, Josh Berkus a écrit : > > > The biggest problem with pgfincore from my point of view is that it > > > only works under Linux, whereas I use a MacOS X machine for my > > > development, and there is also Wind

Re: [HACKERS] Re: proposal: a width specification for s specifier (format function), fix behave when positional and ordered placeholders are used

2013-02-09 Thread Dean Rasheed
> 2013/1/31 Pavel Stehule : >> I am sending rewritten code Nice. I think this will be very useful, and it looks like it now supports everything that printf() does for %s format specifiers, and it's good that %I and %L behave the same. Also the code is looking cleaner. >> It indirect width "*" and

Re: [HACKERS] SP-GiST for ranges based on 2d-mapping and quad-tree

2013-02-09 Thread Alexander Korotkov
Hi! On Mon, Dec 17, 2012 at 2:42 PM, Jeff Davis wrote: > I have attached a patch with some significant edits. > > * In your patch, there was still an inconsistency between the comment > for bounds_adjacent and the code. I refactored it to ensure it always > takes the upper bound as boundA, and t

Re: [HACKERS] Considering Gerrit for CFs

2013-02-09 Thread Simon Riggs
On 8 February 2013 10:23, Magnus Hagander wrote: >> to solve two problems: >> >> 1. maximize the efficiency of existing reviewer time >> >> 2. make tooling not be an obstacle to getting new reviewers > > I think you are missing a fundamental part in this - which is "0. > don't negatively affect t

Re: [HACKERS] function for setting/getting same timestamp during whole transaction

2013-02-09 Thread Miroslav Šimulčík
> I might be confused here, but I think the point is that if you pass > "true" as the third argument to set_config, the setting lasts only for > the duration of the current transaction, like SET LOCAL. Which I > think solves your problem. Yes, but at the end of transaction it is reset to the val

[HACKERS] performance regression in 9.2 CTE with SRF function

2013-02-09 Thread Pavel Stehule
Hello In Czech discussion group was reported performance regression of CTE query. I wrote a test, when I can show it. Execution on 9.2, 9.3 is about 9 sec, on 8.4, 9.1 is about 30ms with default work_mem ='1MB' optimal plan (8.4) QUERY PLAN

[HACKERS] Re: [COMMITTERS] pgsql: psql: Improve expanded print output in tuples-only mode

2013-02-09 Thread Erik Rijkers
On Sat, February 9, 2013 06:24, Peter Eisentraut wrote: > psql: Improve expanded print output in tuples-only mode > > When there are zero result rows, in expanded mode, "(No rows)" is > printed. So far, there was no way to turn this off. Now, when > tuples-only mode is turned on, nothing is print