Re: [HACKERS] Patent warning about the Greenplum source code

2015-10-31 Thread Bruce Momjian
On Sun, Nov 1, 2015 at 01:27:13AM -0500, Bruce Momjian wrote: > On Fri, Oct 30, 2015 at 04:47:35AM -0400, Bruce Momjian wrote: > > Therefore, I caution people from viewing the Greenplum source code as > > you might see patented ideas that could be later implemented in > > Postgres, opening Postgre

Re: [HACKERS] Patent warning about the Greenplum source code

2015-10-31 Thread Bruce Momjian
On Fri, Oct 30, 2015 at 04:47:35AM -0400, Bruce Momjian wrote: > Therefore, I caution people from viewing the Greenplum source code as > you might see patented ideas that could be later implemented in > Postgres, opening Postgres up to increased patent violation problems. I > am also concerned abo

Re: [HACKERS] ParallelContexts can get confused about which worker is which

2015-10-31 Thread Amit Kapila
On Sat, Oct 31, 2015 at 11:35 AM, Robert Haas wrote: > > On Fri, Oct 30, 2015 at 11:24 AM, Robert Haas wrote: > > The other way to fix this is to pass down the index > > that the leader assigns to any given worker, and have the worker use > > that index instead of allocating its own separate inde

Re: [HACKERS] [PATCH] we have added support for box type in SP-GiST index

2015-10-31 Thread Oleg Bartunov
On Sat, Oct 31, 2015 at 9:49 PM, Alexander Lebedev wrote: > Hello, Hacker. > > * [PATCH] add a box index to sp-gist > > We have extended sp-gist with an index that keeps track of boxes > > We have used ideas underlying sp-gist range implementation to > represent 2D boxes as points in 4D spa

Re: [HACKERS] September 2015 Commitfest

2015-10-31 Thread Noah Misch
On Sat, Oct 31, 2015 at 03:43:13PM +, Nathan Wagner wrote: > There was then some further discussion on the interface, and what to do > with startup files, and nothing was really decided, and then the thread > petered out. This potential reviewer is then left with the conclusion > that this pat

[HACKERS] [PATCH] we have added support for box type in SP-GiST index

2015-10-31 Thread Alexander Lebedev
Hello, Hacker. * [PATCH] add a box index to sp-gist We have extended sp-gist with an index that keeps track of boxes We have used ideas underlying sp-gist range implementation to represent 2D boxes as points in 4D space. We use quad tree analogue, but in 4-dimensional space. We call thi

[HACKERS] Minor clarifying changes to abbreviated key abort code comments

2015-10-31 Thread Peter Geoghegan
Attached are a couple of patches that only change code comments. The first (abort abbreviation) patch is recommended for backpatch to 9.5. The second is a tiny tweak. -- Peter Geoghegan From b49bc2ce0b83bc5e4c33c5728f885058ab52cdae Mon Sep 17 00:00:00 2001 From: Peter Geoghegan Date: Sat, 31 Oct

Re: [HACKERS] proposal: multiple psql option -c

2015-10-31 Thread Pavel Stehule
Hi 2015-09-21 16:46 GMT+02:00 Adam Brightwell < adam.brightw...@crunchydatasolutions.com>: > Pavel, > > > with -1 option support > > FWIW, I have tried to apply this patch against master (7f11724) and > there is a minor error, see below. > > From patch: > > patching file src/bin/psql/settings.h >

Re: [HACKERS] [BUGS] BUG #12989: pg_size_pretty with negative values

2015-10-31 Thread Julien Rouhaud
On 20/09/2015 14:16, Adrian.Vondendriesch wrote: > Hi all, > Hello, > Am 06.04.2015 um 20:52 schrieb Tom Lane: >> "David G. Johnston" writes: >>> I'll let a hacker determine whether this is a bug or a feature >>> request though it is a POLA violation in either case. >> >> I'd say it's a featur

[HACKERS] COPY (INSERT/UPDATE/DELETE .. RETURNING ..)

2015-10-31 Thread Marko Tiikkaja
Hi, Attached is a patch for being able to do $SUBJECT without a CTE. The reasons this is better than a CTE version are: 1) It's not obvious why a CTE version works but a plain one doesn't 2) This one has less overhead (I measured a ~12% improvement on a not-too-unreasonable test case)

Re: [HACKERS] pg_stat_statements query jumbling question

2015-10-31 Thread Julien Rouhaud
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Hello, On 10/10/2015 08:46, Satoshi Nagayasu wrote: > On 2015/10/03 6:18, Peter Geoghegan wrote: >> On Wed, Sep 2, 2015 at 7:41 PM, Satoshi Nagayasu >> wrote: >>> I know this still needs to be discussed, but I would like to >>> submit a patch for fur

Re: [HACKERS] snapshots in analyze

2015-10-31 Thread Andres Freund
On 2015-10-31 12:52:44 -0400, Tom Lane wrote: > Andres Freund writes: > > Another angle would be trying to reduce the effects of longrunning > > transaction. Right now holding a snapshot open for 100 seconds results > > in profiles like this: ... > > which is pretty extreme. It's not such a seldom

Re: [HACKERS] September 2015 Commitfest

2015-10-31 Thread Marko Tiikkaja
On 10/31/15 12:42 AM, Michael Paquier wrote: So, seeing nothing happening I have done the above, opened 2015-11 CF and closed the current one. Are we doing these in an Australian time zone now? It was quite unpleasant to find that the 2015-11 is "in progress" already and two of my patches wi

Re: [HACKERS] snapshots in analyze

2015-10-31 Thread Tom Lane
Andres Freund writes: > Another angle would be trying to reduce the effects of longrunning > transaction. Right now holding a snapshot open for 100 seconds results > in profiles like this: ... > which is pretty extreme. It's not such a seldom thing to hold a snapshot > (e.g. pg_dump...) open for a

Re: [HACKERS] September 2015 Commitfest

2015-10-31 Thread Tom Lane
[ separate response for questions that are about process not mechanics ] Nathan Wagner writes: > Back to the patch in question, so Mr Brightwell noted that the patch > doesn't apply against master. Should someone then mark the patch as > waiting on author? Is failing to apply against master a '

Re: [HACKERS] September 2015 Commitfest

2015-10-31 Thread Nathan Wagner
On Sat, Oct 31, 2015 at 12:08:58PM -0400, Tom Lane wrote: > Nathan Wagner writes: > > Second, it would be convenient if there were a make target that would > > set up a test environment. Effectively do what the 'make check' does, > > but don't run the tests and leave the database up. It should p

Re: [HACKERS] September 2015 Commitfest

2015-10-31 Thread Tom Lane
Nathan Wagner writes: > Second, it would be convenient if there were a make target that would > set up a test environment. Effectively do what the 'make check' does, > but don't run the tests and leave the database up. It should probably > drop you into a shell that has the paths set up as well.

Re: [HACKERS] September 2015 Commitfest

2015-10-31 Thread Nathan Wagner
On Sat, Oct 31, 2015 at 08:03:59AM +0100, Robert Haas wrote: > +1. FWIW, I'm willing to review some patches for this CommitFest, but > if the committers have to do first-round review as well as > committer-review of every patch in the CommitFest, this is going to be > long, ugly, and painful. We

[HACKERS] snapshots in analyze

2015-10-31 Thread Andres Freund
Hi, I previously complained about analyze keeping a snapshot while running in: http://archives.postgresql.org/message-id/20141018174909.GA5790%40alap3.anarazel.de since then I've been bitten by that, and I've seen other people being bitten by it. on a scale 400 database (so analyze actually take

Re: [HACKERS] ALTER ... OWNER TO ... vs. ALTER DEFAULT PRIVILEGES

2015-10-31 Thread David Fetter
On Sat, Oct 31, 2015 at 12:16:31AM +0100, Robert Haas wrote: > On Thu, Oct 29, 2015 at 10:31 PM, David Fetter wrote: > > Had this been part of the original ALTER DEFAULT PRIVILEGES patch, > > those privileges would simply have been applied. Since it wasn't, I'm > > ass-u-me'ing that changing the

Re: [HACKERS] Freeze avoidance of very large table.

2015-10-31 Thread Amit Kapila
On Fri, Oct 30, 2015 at 6:03 AM, Masahiko Sawada wrote: > > > > v20 patch has a bug in result of regression test. > Attached updated v21 patch. > Couple of more review comments: -- 1. @@ -615,6 +617,8 @@ typedef struct PgStat_StatTabEntry PgS

[HACKERS] eXtensible Transaction Manager API

2015-10-31 Thread konstantin knizhnik
Hi, PostgresPro cluster team wants to announce proposal for eXtensible Transaction Manager API and reference implementation of distributed transaction manager (pg_dtm). pg_dtm is just a standard PostgreSQL extension which should be installed in normal way. Source of pg_dtm and PostgreSQL patc

Re: [HACKERS] pg_dump LOCK TABLE ONLY question

2015-10-31 Thread Simon Riggs
On 2 October 2015 at 01:19, Michael Paquier wrote: > On Thu, Oct 1, 2015 at 10:43 PM, Filip RembiaƂkowski > wrote: > > I just want to understand why there is LOCK TABLE not LOCK TABLE ONLY. > > It seems to me that you'd still want to use LOCK TABLE particularly if > the dump is only done on a su

Re: [HACKERS] September 2015 Commitfest

2015-10-31 Thread Andres Freund
On 2015-10-31 06:50:09 +0100, Michael Paquier wrote: > On Sat, Oct 31, 2015 at 12:55 AM, Andres Freund wrote: > > You seemingly moved all entries, even the ones which were > > waiting-on-author for a long while, over? I think we should return items > > on there with lot of prejudice. Otherwise we'

Re: [HACKERS] September 2015 Commitfest

2015-10-31 Thread Robert Haas
On Sat, Oct 31, 2015 at 6:50 AM, Michael Paquier wrote: > I know. We should normally begin the cleanup activity far earlier IMO, > like at the end of the commit fest month to give patch authors a > couple of weeks to rework what they have if they would like to resend > something for the next commi