Re: [HACKERS] dblink: add polymorphic functions.

2015-07-05 Thread Corey Huinker
On Sun, Jul 5, 2015 at 9:00 PM, Joe Conway wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > On 07/05/2015 12:25 PM, Joe Conway wrote: > > On 02/22/2015 10:26 PM, Corey Huinker wrote: > >> Changes in this patch: - added polymorphic versions of > >> dblink_fetch() - upped dblink version

Re: [HACKERS] C# reading result from a function

2015-07-05 Thread Ashutosh Bapat
This mail is probably appropriate for some C# mailing list. I am not familiar with C#, but in the above code, I do not see where is test_func() being called? On Sat, Jul 4, 2015 at 1:44 PM, drunken wrote: > Hi guys, I have a problem with a sipmle function in postgresql. The func() > calculates

Re: [HACKERS] pg_archivecleanup, and backup filename to specify as an argument

2015-07-05 Thread Michael Paquier
On Mon, Jul 6, 2015 at 2:06 PM, Fujii Masao wrote: > Barring any objection, I will remove the warning from the document. Nuke it. -- Michael -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hack

Re: [HACKERS] Parallel Seq Scan

2015-07-05 Thread Amit Kapila
On Mon, Jul 6, 2015 at 3:26 AM, Jeff Davis wrote: > > On Fri, 2015-07-03 at 17:35 +0530, Amit Kapila wrote: > > > Attached, find the rebased version of patch. > > > > Comments: > > > * The heapam.c changes seem a little ad-hoc. Conceptually, which > portions should be affected by parallelism? How

Re: [HACKERS] pg_archivecleanup, and backup filename to specify as an argument

2015-07-05 Thread Fujii Masao
On Fri, Jul 3, 2015 at 9:56 PM, Fujii Masao wrote: > On Thu, Jul 2, 2015 at 8:02 PM, Fujii Masao wrote: >> Hi, >> >> While I'm implementing the patch around pg_archivecleanup, I found >> the following warning about pg_archivecleanup in the document. >> >> Note that the .backup file name passe

Re: [HACKERS] Inconsistent style in pgbench's error messages

2015-07-05 Thread Fabien COELHO
Hello Tom, Personally I think "pgbench: " adds very little and so I'd rather get rid of it, but I'm sure others feel differently. I think that one of the reason for this is that once pgbench started to run it is desirable to differentiate error messages that come from libpq and those that c

Re: [HACKERS] Let PostgreSQL's On Schedule checkpoint write buffer smooth spread cycle by tuning IsCheckpointOnSchedule?

2015-07-05 Thread Amit Kapila
On Sun, Jul 5, 2015 at 1:18 PM, Heikki Linnakangas wrote: > On 07/04/2015 07:34 PM, Fabien COELHO wrote: > >> I have ran some tests with this patch and the detailed results of the >>> runs are attached with this mail. >>> >> >> I do not understand really the aggregated figures in the files attach

Re: [HACKERS] Memory Accounting v11

2015-07-05 Thread David Rowley
On 15 June 2015 at 07:43, Jeff Davis wrote: > > * There was a slowdown reported of around 1-3% (depending on the exact > version of the patch) on an IBM power machine when doing an index > rebuild. The results were fairly noisy for me, but it seemed to hold up. > See http://www.postgresql.org/mes

Re: [HACKERS] Fix broken Install.bat when target directory contains a space

2015-07-05 Thread Michael Paquier
On Sat, Jul 4, 2015 at 3:57 AM, Heikki Linnakangas wrote: > Hmm. Why is install.bat not like build.bat, i.e. just a thin wrapper that > just calls install.pl, passing all arguments? I guess we just haven't noticed it. And indeed it makes everything more simple, and fixes as well the error reported

Re: [HACKERS] dblink: add polymorphic functions.

2015-07-05 Thread Joe Conway
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 07/05/2015 12:25 PM, Joe Conway wrote: > On 02/22/2015 10:26 PM, Corey Huinker wrote: >> Changes in this patch: - added polymorphic versions of >> dblink_fetch() - upped dblink version # to 1.2 because of new >> functions - migration 1.1 -> 1.2 -

Re: [HACKERS] Foreign join pushdown vs EvalPlanQual

2015-07-05 Thread Kouhei Kaigai
> > Also, I don't want to stick on the assumption that relations involved in > > remote join are all managed by same foreign-server no longer. > > The following two ideas introduce possible enhancement of remote join > > feature that involved local relations; replicated table or transformed > > to

Re: [HACKERS] Inconsistent style in pgbench's error messages

2015-07-05 Thread Joshua D. Drake
On 07/05/2015 04:46 PM, Tom Lane wrote: I made a pass over pgbench's error messages to try to make them meet project style guidelines. There was one rather large bit of inconsistency that I didn't try to fix, though: something like half of the messages prepend "pgbench: " to the front, but the

[HACKERS] Inconsistent style in pgbench's error messages

2015-07-05 Thread Tom Lane
I made a pass over pgbench's error messages to try to make them meet project style guidelines. There was one rather large bit of inconsistency that I didn't try to fix, though: something like half of the messages prepend "pgbench: " to the front, but the other half don't. Worse yet, some of the o

Re: [HACKERS] raw output from copy

2015-07-05 Thread Pavel Stehule
2015-07-02 17:02 GMT+02:00 Tom Lane : > Andrew Dunstan writes: > > Does the COPY line protocol even support binary data? > > The protocol, per se, just transmits a byte stream. There is a field > in the CopyInResponse/CopyOutResponse messages that indicates whether > a text or binary copy is bei

Re: [HACKERS] Parallel Seq Scan

2015-07-05 Thread Jeff Davis
On Fri, 2015-07-03 at 17:35 +0530, Amit Kapila wrote: > Attached, find the rebased version of patch. > Comments: * The heapam.c changes seem a little ad-hoc. Conceptually, which portions should be affected by parallelism? How do we know we didn't miss something? * Why is initscan getting the n

Re: [HACKERS] PostgreSQL 9.5 Alpha 1 build fail with perl 5.22

2015-07-05 Thread Andrew Dunstan
On 07/04/2015 11:02 AM, Tom Lane wrote: I wondered how come we had not seen this problem in the buildfarm, but the answer appears to be that our only working Cygwin critter (brolga) doesn't build any of the optional PLs, so it skips these modules altogether. Seems like we need to improve that

Re: [HACKERS] dblink: add polymorphic functions - review

2015-07-05 Thread Joe Conway
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 06/18/2015 09:57 PM, Pavel Stehule wrote: > 2015-06-19 6:32 GMT+02:00 Thomas Munro > >: > > On Fri, Jun 19, 2015 at 4:17 PM, Pavel Stehule > mailto:pavel.steh...@gmail.com>> wrote: >> I am sorry. I didn't the

Re: [HACKERS] dblink: add polymorphic functions.

2015-07-05 Thread Joe Conway
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 02/22/2015 10:26 PM, Corey Huinker wrote: > Changes in this patch: - added polymorphic versions of > dblink_fetch() - upped dblink version # to 1.2 because of new > functions - migration 1.1 -> 1.2 - DocBook changes for dblink(), > dblink_get_result

Re: [HACKERS] Rework the way multixact truncations work

2015-07-05 Thread Andres Freund
On July 5, 2015 8:50:57 PM GMT+02:00, Robert Haas wrote: >On Sun, Jul 5, 2015 at 2:28 PM, Andres Freund >wrote: >> (quick answer, off now) >> >> On 2015-07-05 14:20:11 -0400, Robert Haas wrote: >>> On Thu, Jul 2, 2015 at 2:28 PM, Andres Freund >wrote: >>> > On 2015-07-02 13:58:45 -0400, Robert H

Re: [HACKERS] Rework the way multixact truncations work

2015-07-05 Thread Robert Haas
On Sun, Jul 5, 2015 at 2:28 PM, Andres Freund wrote: > (quick answer, off now) > > On 2015-07-05 14:20:11 -0400, Robert Haas wrote: >> On Thu, Jul 2, 2015 at 2:28 PM, Andres Freund wrote: >> > On 2015-07-02 13:58:45 -0400, Robert Haas wrote: >> >> I seriously, seriously doubt that it is a good id

Re: [HACKERS] Rework the way multixact truncations work

2015-07-05 Thread Andres Freund
(quick answer, off now) On 2015-07-05 14:20:11 -0400, Robert Haas wrote: > On Thu, Jul 2, 2015 at 2:28 PM, Andres Freund wrote: > > On 2015-07-02 13:58:45 -0400, Robert Haas wrote: > >> I seriously, seriously doubt that it is a good idea to perform the > >> legacy truncation from MultiXactAdvance

Re: [HACKERS] Rework the way multixact truncations work

2015-07-05 Thread Robert Haas
On Thu, Jul 2, 2015 at 2:28 PM, Andres Freund wrote: > On 2015-07-02 13:58:45 -0400, Robert Haas wrote: >> On Thu, Jul 2, 2015 at 11:52 AM, Robert Haas wrote: >> > Will look at 0003 next. >> >> +appendStringInfo(buf, "offsets [%u, %u), members [%u, %u)", >> >> I don't think we typically u

Re: [HACKERS] [patch] Simple Typo patch.

2015-07-05 Thread Tom Lane
CharSyam writes: > I just found simple typo error :) > statment -> statement > excercise -> exercise > They are all in just comments :) Pushed, thanks. (For future reference, a patch like this needs to update the expected-output files too.) regards, tom lane -- Sent v

[HACKERS] [patch] Simple Typo patch.

2015-07-05 Thread CharSyam
I just found simple typo error :) statment -> statement excercise -> exercise They are all in just comments :) typos.patch Description: Binary data -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pg

Re: [HACKERS] Comfortably check BackendPID with psql

2015-07-05 Thread Petr Korobeinikov
+1 for Julien's patch.

Re: [HACKERS] auto_explain sample rate

2015-07-05 Thread Julien Rouhaud
On 03/06/2015 15:00, Craig Ringer wrote: > > > On 3 June 2015 at 20:04, Andres Freund > wrote: > > On 2015-06-03 18:54:24 +0800, Craig Ringer wrote: > > OK, here we go. > > Hm. Wouldn't random sampling be better than what you do? If your queries > hav

Re: [HACKERS] [patch] typo in brin.sql

2015-07-05 Thread Tom Lane
Christoph Berg writes: > There was a stray "s" in "classes implement*s*". I've also added a > "the" to make the sentence more readable (at least for me). Pushed, thanks. (I also tweaked the second part of that sentence a bit, though I'm not sure it's 100% perfect grammar even yet.)

Re: [HACKERS] Let PostgreSQL's On Schedule checkpoint write buffer smooth spread cycle by tuning IsCheckpointOnSchedule?

2015-07-05 Thread Fabien COELHO
No, I'm not saying anyone *has* to do anything. What I'm saying is that I'm not convinced by your analysis. Well, the gist of my analysis is really to say that there are potential performance issues with the proposed change, and that it must be tested thoroughly. The details may varry:-)

Re: [HACKERS] Let PostgreSQL's On Schedule checkpoint write buffer smooth spread cycle by tuning IsCheckpointOnSchedule?

2015-07-05 Thread Andres Freund
On 2015-07-05 11:05:28 -0400, Robert Haas wrote: > More broadly, I don't really know how to test this patch and show when > it helps and when it hurts. And I think we need that, rather than > just a theoretical analysis, to tune the behavior. Heikki, can you > describe what you think a good test

Re: [HACKERS] Exposing PG_VERSION_NUM in pg_config

2015-07-05 Thread Pavel Stehule
2015-07-05 16:51 GMT+02:00 Tom Lane : > Michael Paquier writes: > > On Fri, Jul 3, 2015 at 6:27 AM, Tom Lane wrote: > >> Michael Paquier writes: > >>> ... So attached is a patch that adds VERSION_NUM in > >>> Makefile.global. > > >> While there was not exactly universal consensus that we need t

Re: [HACKERS] Exposing PG_VERSION_NUM in pg_config

2015-07-05 Thread Andres Freund
On 2015-07-05 10:51:48 -0400, Tom Lane wrote: > Any objections to doing that? Nope. -- 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] Let PostgreSQL's On Schedule checkpoint write buffer smooth spread cycle by tuning IsCheckpointOnSchedule?

2015-07-05 Thread Robert Haas
On Sun, Jul 5, 2015 at 1:19 AM, Fabien COELHO wrote: > Do you mean I have to proove that there is an actual problem induced from > this patch? No, I'm not saying anyone *has* to do anything. What I'm saying is that I'm not convinced by your analysis. I don't think we have enough evidence at thi

Re: [HACKERS] Exposing PG_VERSION_NUM in pg_config

2015-07-05 Thread Tom Lane
Michael Paquier writes: > On Fri, Jul 3, 2015 at 6:27 AM, Tom Lane wrote: >> Michael Paquier writes: >>> ... So attached is a patch that adds VERSION_NUM in >>> Makefile.global. >> While there was not exactly universal consensus that we need this, the >> patch as given is merely two lines, so i

Re: [HACKERS] Let PostgreSQL's On Schedule checkpoint write buffer smooth spread cycle by tuning IsCheckpointOnSchedule?

2015-07-05 Thread Fabien COELHO
Hello Heikki, I think that the load is distributed as the derivative of this function, that is (1.5 * x ** 0.5): It starts at 0 but very quicky reaches 0.5, it pass the 1.0 (average load) around 40% progress, and ends up at 1.5, that is the finishing load is 1.5 the average load, just before fs

Re: [HACKERS] Let PostgreSQL's On Schedule checkpoint write buffer smooth spread cycle by tuning IsCheckpointOnSchedule?

2015-07-05 Thread Fabien COELHO
You don't have to do anything if you don't want to. Sure:-) What I mean is that I think that this patch is not ripe, and I understood that some people were suggesting that it could be applied as is right away. I'm really disagreeing with that. I said myself that this needs performance test

Re: [HACKERS] Comfortably check BackendPID with psql

2015-07-05 Thread Julien Rouhaud
On 12/06/2015 06:56, Noah Misch wrote: > On Thu, Jun 11, 2015 at 04:05:13PM -0500, Jim Nasby wrote: >> On 6/11/15 4:55 AM, Andres Freund wrote: >>> On 2015-06-11 09:41:17 +, Naoya Anzai wrote: This is a so tiny patch but I think it is very useful for hackers and DBAs. When we debug wi

[HACKERS] [patch] typo in brin.sql

2015-07-05 Thread Christoph Berg
There was a stray "s" in "classes implement*s*". I've also added a "the" to make the sentence more readable (at least for me). Christoph -- c...@df7cb.de | http://www.df7cb.de/ diff --git a/doc/src/sgml/brin.sgml b/doc/src/sgml/brin.sgml new file mode 100644 index e25f09c..894c269 *** a/doc/src/s

Re: [HACKERS] Let PostgreSQL's On Schedule checkpoint write buffer smooth spread cycle by tuning IsCheckpointOnSchedule?

2015-07-05 Thread Heikki Linnakangas
On 07/05/2015 08:19 AM, Fabien COELHO wrote: I am a bit skeptical about this. We need test scenarios that clearly show the benefit of having and of not having this behavior. It might be that doing this always is fine for everyone. Do you mean I have to proove that there is an actual problem in

Re: [HACKERS] Let PostgreSQL's On Schedule checkpoint write buffer smooth spread cycle by tuning IsCheckpointOnSchedule?

2015-07-05 Thread Heikki Linnakangas
On 07/04/2015 07:34 PM, Fabien COELHO wrote: In summary, the X^1.5 correction seems to work pretty well. It doesn't completely eliminate the problem, but it makes it a lot better. I've looked at the maths. I think that the load is distributed as the derivative of this function, that is (1.5