Re: MSVC Build support with visual studio 2019

2019-06-27 Thread Michael Paquier
On Wed, Jun 26, 2019 at 10:29:05PM +1000, Haribabu Kommi wrote: > Thanks for the review. Yes, that patch applies till 9.5, it is my mistake > in naming the patch. I have been able to finally set up an environment with VS 2019 (as usual this stuff needs time, anyway..), and I can confirm that the p

RE: [patch]socket_timeout in interfaces/libpq

2019-06-27 Thread nagaura.ryo...@fujitsu.com
Hi, Michael-san. > From: Michael Paquier > On Wed, Jun 26, 2019 at 04:13:36AM +, nagaura.ryo...@fujitsu.com wrote: > > I don't think that the rest one of my proposals has been rejected > > completely, so I want to restart discussion. > I recall on the matter that there was consensus that nobo

Re: FETCH FIRST clause PERCENT option

2019-06-27 Thread Surafel Temesgen
Hello, The attached patch include the fix for all the comment given regards Surafel On Mon, Apr 1, 2019 at 12:34 AM Andres Freund wrote: > Hi, > > On 2019-03-29 12:04:50 +0300, Surafel Temesgen wrote: > > > + if (node->limitOption == PERCENTAGE) > > +

Extra quote_all_identifiers in _dumpOptions

2019-06-27 Thread Arthur Zakirov
Hello hackers, While working on pg_dump I noticed the extra quote_all_identifiers in _dumpOptions struct. I attached the patch. It came from refactoring by 0eea8047bf. There is also a discussion: https://www.postgresql.org/message-id/CACw0%2B13ZUcXbj9GKJNGZTkym1SXhwRu28nxHoJMoX5Qwmbg_%2Bw%40ma

Re: GiST "choose subtree" support function to inline penalty

2019-06-27 Thread Komяpa
On Thu, Jun 27, 2019 at 6:00 AM Tom Lane wrote: > =?UTF-8?Q?Darafei_=22Kom=D1=8Fpa=22_Praliaskouski?= > writes: > > I'm looking at PostGIS geometry GiST index build times and try to > optimize > > withing the current GiST framework. The function that shows a lot on my > > flame graphs is penalty

Obsolete comment in commands/analyze.c

2019-06-27 Thread Etsuro Fujita
Hi, I think commit 83e176ec1, which moved block sampling functions to utils/misc/sampling.c, forgot to update this comment in commands/analyze.c: "This algorithm is from Jeff Vitter's paper (see full citation below)"; since the citation was also moved to utils/misc/sampling.c, I think the "see ful

Re: mcvstats serialization code is still shy of a load

2019-06-27 Thread Tomas Vondra
On Thu, Jun 27, 2019 at 12:04:30AM -0400, Tom Lane wrote: Tomas Vondra writes: OK. Attached is a patch ditching the alignment in serialized data. I've ditched the macros to access parts of serialized data, and everything gets copied. I lack energy to actually read this patch right now, and I

Re: GiST VACUUM

2019-06-27 Thread Heikki Linnakangas
On 26/06/2019 06:07, Thomas Munro wrote: On Wed, Jun 26, 2019 at 2:33 PM Michael Paquier wrote: On Tue, Jun 25, 2019 at 02:38:43PM +0500, Andrey Borodin wrote: I feel a little uncomfortable with number 0x7fff right in code. PG_INT32_MAX... MaxTransactionId / 2? Yeah, that makes sense

Use relative rpath if possible

2019-06-27 Thread Peter Eisentraut
On several popular operating systems, we can use relative rpaths, using the $ORIGIN placeholder, so that the resulting installation is relocatable. Then we also don't need to set LD_LIBRARY_PATH during make check. This implementation will use a relative rpath if bindir and libdir are under the sa

Re: Obsolete comment in commands/analyze.c

2019-06-27 Thread Daniel Gustafsson
> On 27 Jun 2019, at 13:05, Etsuro Fujita wrote: > since the citation was also moved to > utils/misc/sampling.c, I think the "see full citation below" part > should be updated accordingly. Attached is a patch for that. Agreed, nice catch! cheers ./daniel

Re: GiST VACUUM

2019-06-27 Thread Andrey Borodin
> 27 июня 2019 г., в 16:38, Heikki Linnakangas написал(а): > > I haven't done any testing on this yet. Andrey, would you happen to have an > environment ready to test this? Thanks! I will do some testing this evening (UTC+5). But I'm not sure I can reliably test wraparound of xids... I wil

Re: PG 11 JIT deform failure

2019-06-27 Thread didier
Hi, I searched the mailing list but found nothing. Any reason why TupleDescAttr is a macro and not a static inline? Rather than adding an Assert attached POC replace TupleDescAttr macro by a static inline function with AssertArg. It: - Factorize Assert. - Trigger an Assert in JIT_deform if natts

Re: SQL/JSON path issues/questions

2019-06-27 Thread Thom Brown
On Wed, 19 Jun 2019 at 20:04, Alexander Korotkov wrote: > > On Wed, Jun 19, 2019 at 7:07 PM Thom Brown wrote: > > On Thu, 13 Jun 2019 at 14:59, Thom Brown wrote: > > > > > > Hi, > > > > > > I've been reading through the documentation regarding jsonpath and > > > jsonb_path_query etc., and I have

OpenSSL specific value under USE_SSL instead of USE_OPENSSL

2019-06-27 Thread Daniel Gustafsson
The ssl_ciphers GUC for configuring cipher suites sets the default value based on USE_SSL but it’s actually an OpenSSL specific value. As with other such changes, it works fine now but will become an issue should we support other TLS backends. Attached patch fixes this. cheers ./daniel opens

pg_receivewal documentation

2019-06-27 Thread Jesper Pedersen
Hi, Here is a patch for the pg_receivewal documentation to highlight that WAL isn't acknowledged to be applied. I'll add a CF entry for it. Best regards, Jesper >From 138e09c74db5ea08fd03b4e77853e2ca01742512 Mon Sep 17 00:00:00 2001 From: jesperpedersen Date: Thu, 27 Jun 2019 09:54:44 -0400

Re: [HACKERS] Restricting maximum keep segments by repslots

2019-06-27 Thread Jehan-Guillaume de Rorthais
Hi all, Being interested by this feature, I did a patch review. This features adds the GUC "max_slot_wal_keep_size". This is the maximum amount of WAL that can be kept in "pg_wal" by active slots. If the amount of WAL is superior to this limit, the slot is deactivated and its status (new filed i

Missing PG 12 stable branch

2019-06-27 Thread Bruce Momjian
Does anyone know why there is no PG 12 stable branch in our git tree? $ git branch -l REL9_4_STABLE REL9_5_STABLE REL9_6_STABLE REL_10_STABLE REL_11_STABLE master They exist for earlier releases. Is this a problem? -- Bruce

Re: Missing PG 12 stable branch

2019-06-27 Thread Bruce Momjian
On Thu, Jun 27, 2019 at 10:28:40AM -0400, Bruce Momjian wrote: > Does anyone know why there is no PG 12 stable branch in our git tree? > > $ git branch -l > REL9_4_STABLE > REL9_5_STABLE > REL9_6_STABLE > REL_10_STABLE > REL_11_STABLE > master

Re: Missing PG 12 stable branch

2019-06-27 Thread Tom Lane
Bruce Momjian writes: > Does anyone know why there is no PG 12 stable branch in our git tree? For the record, I'm intending to make the branch as soon as the July CF starts (i.e., first thing Monday). regards, tom lane

Re: Missing PG 12 stable branch

2019-06-27 Thread Tomas Vondra
On Thu, Jun 27, 2019 at 10:28:40AM -0400, Bruce Momjian wrote: Does anyone know why there is no PG 12 stable branch in our git tree? $ git branch -l REL9_4_STABLE REL9_5_STABLE REL9_6_STABLE REL_10_STABLE REL_11_STABLE master T

Re: Obsolete comment in commands/analyze.c

2019-06-27 Thread Tomas Vondra
On Thu, Jun 27, 2019 at 01:53:52PM +0200, Daniel Gustafsson wrote: On 27 Jun 2019, at 13:05, Etsuro Fujita wrote: since the citation was also moved to utils/misc/sampling.c, I think the "see full citation below" part should be updated accordingly. Attached is a patch for that. Agreed, nice

Re: Fix doc bug in logical replication.

2019-06-27 Thread Tomas Vondra
On Sun, Jun 23, 2019 at 10:26:47PM -0400, Robert Treat wrote: On Sun, Jun 23, 2019 at 1:25 PM Peter Eisentraut wrote: On 2019-04-12 19:52, Robert Treat wrote: > It is clear to me that the docs are wrong, but I don't see anything > inherently incorrect about the code itself. Do you have suggest

Re: UCT (Re: pgsql: Update time zone data files to tzdata release 2019a.)

2019-06-27 Thread Tom Lane
Andrew Gierth writes: > "Tom" == Tom Lane writes: > Tom> In general, the point I'm trying to make is that our policy should > Tom> be "Ties are broken arbitrarily, and if you don't like the choice > Tom> that initdb makes, here's how to fix it". > Yes, you've repeated that point at some lengt

Re: GiST VACUUM

2019-06-27 Thread Andrey Borodin
> 27 июня 2019 г., в 16:38, Heikki Linnakangas написал(а): > > I haven't done any testing on this yet. Andrey, would you happen to have an > environment ready to test this? Patches do not deadlock and delete pages on "rescan test" - setup that we used to detect "left jumps" in during develo

Re: UCT (Re: pgsql: Update time zone data files to tzdata release 2019a.)

2019-06-27 Thread Robert Haas
On Tue, Jun 25, 2019 at 8:18 PM Tom Lane wrote: > As long as we have a trivial and obviously apolitical rule like > alphabetical order, I think we can skate over such things; but the minute > we have any sort of human choices involved there, we're going to be > getting politically driven requests

Re: [HACKERS] Regression tests vs existing users in an installation

2019-06-27 Thread Robert Haas
On Tue, Jun 25, 2019 at 11:33 PM Tom Lane wrote: > Comments? LGTM. s/must/should/ ? -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company

Re: Fix doc bug in logical replication.

2019-06-27 Thread Dave Cramer
On Thu, 27 Jun 2019 at 12:50, Tomas Vondra wrote: > On Sun, Jun 23, 2019 at 10:26:47PM -0400, Robert Treat wrote: > >On Sun, Jun 23, 2019 at 1:25 PM Peter Eisentraut > > wrote: > >> > >> On 2019-04-12 19:52, Robert Treat wrote: > >> > It is clear to me that the docs are wrong, but I don't see any

Re: UCT (Re: pgsql: Update time zone data files to tzdata release 2019a.)

2019-06-27 Thread Tom Lane
Robert Haas writes: > I'm kind of unsure what to think about this whole debate > substantively. If Andrew is correct that zone.tab or zone1970.tab is a > list of time zone names to be preferred over alternatives, then it > seems like we ought to prefer them. It's not really clear to me that the I

Hypothetical indexes using BRIN broken since pg10

2019-06-27 Thread Julien Rouhaud
Hello, I just realized that 7e534adcdc7 broke support for hypothetical indexes using BRIN am. Attached patch fix the issue. There's no interface to provide the hypothetical pagesPerRange value, so I used the default one, and used simple estimates. I'll add this patch to the next commitfest. dif

Re: POC: Cleaning up orphaned files using undo logs

2019-06-27 Thread Robert Haas
On Tue, Jun 25, 2019 at 4:00 PM Amit Kapila wrote: > [ new patches ] I happened to open up 0001 from this series, which is from Thomas, and I do not think that the pg_buffercache changes are correct. The idea here is that the customer might install version 1.3 or any prior version on an old rele

Re: Hypothetical indexes using BRIN broken since pg10

2019-06-27 Thread Alvaro Herrera
Hi, thanks for the patch. On 2019-Jun-27, Julien Rouhaud wrote: > I just realized that 7e534adcdc7 broke support for hypothetical > indexes using BRIN am. Attached patch fix the issue. > > There's no interface to provide the hypothetical pagesPerRange value, > so I used the default one, and use

Re: Fix doc bug in logical replication.

2019-06-27 Thread Tomas Vondra
On Thu, Jun 27, 2019 at 01:46:47PM -0400, Dave Cramer wrote: On Thu, 27 Jun 2019 at 12:50, Tomas Vondra wrote: On Sun, Jun 23, 2019 at 10:26:47PM -0400, Robert Treat wrote: >On Sun, Jun 23, 2019 at 1:25 PM Peter Eisentraut > wrote: >> >> On 2019-04-12 19:52, Robert Treat wrote: >> > It is clea

Re: Hypothetical indexes using BRIN broken since pg10

2019-06-27 Thread Julien Rouhaud
On Thu, Jun 27, 2019 at 8:14 PM Alvaro Herrera wrote: > > Hi, thanks for the patch. Thanks for looking at it! > On 2019-Jun-27, Julien Rouhaud wrote: > > > I just realized that 7e534adcdc7 broke support for hypothetical > > indexes using BRIN am. Attached patch fix the issue. > > > > There's no

Re: pglz performance

2019-06-27 Thread Andrey Borodin
> 13 мая 2019 г., в 12:14, Michael Paquier написал(а): > > Decompression can matter a lot for mostly-read workloads and > compression can become a bottleneck for heavy-insert loads, so > improving compression or decompression should be two separate > problems, not two problems linked. Any impr

Re: Fix doc bug in logical replication.

2019-06-27 Thread Dave Cramer
On Thu, 27 Jun 2019 at 14:20, Tomas Vondra wrote: > On Thu, Jun 27, 2019 at 01:46:47PM -0400, Dave Cramer wrote: > >On Thu, 27 Jun 2019 at 12:50, Tomas Vondra > >wrote: > > > >> On Sun, Jun 23, 2019 at 10:26:47PM -0400, Robert Treat wrote: > >> >On Sun, Jun 23, 2019 at 1:25 PM Peter Eisentraut >

Re: Hypothetical indexes using BRIN broken since pg10

2019-06-27 Thread Alvaro Herrera
On 2019-Jun-27, Julien Rouhaud wrote: > On Thu, Jun 27, 2019 at 8:14 PM Alvaro Herrera > wrote: > > I think it would look nicer to have a routine parallel to brinGetStats() > > (brinGetStatsHypothetical?), instead of polluting selfuncs.c with these > > gory details. > > I'm not opposed to it,

Re: Hypothetical indexes using BRIN broken since pg10

2019-06-27 Thread Tom Lane
Alvaro Herrera writes: > On 2019-Jun-27, Julien Rouhaud wrote: >> On Thu, Jun 27, 2019 at 8:14 PM Alvaro Herrera >> wrote: >>> I think it would look nicer to have a routine parallel to brinGetStats() >>> (brinGetStatsHypothetical?), instead of polluting selfuncs.c with these >>> gory details. >

Missing hook for UPPERREL_PARTIAL_GROUP_AGG rels?

2019-06-27 Thread Erik Nordström
Hi, I noticed that the create_upper_paths_hook is never called for partially grouped rels. Is this intentional or a bug? Only the FDW routine's GetForeignUpperPaths hook is called for partially grouped rels. This seems odd since the regular create_upper_paths_hook gets called for all other upper r

Re: Missing hook for UPPERREL_PARTIAL_GROUP_AGG rels?

2019-06-27 Thread Tom Lane
=?UTF-8?Q?Erik_Nordstr=C3=B6m?= writes: > I noticed that the create_upper_paths_hook is never called for partially > grouped rels. Is this intentional or a bug? Only the FDW routine's > GetForeignUpperPaths hook is called for partially grouped rels. This seems > odd since the regular create_upper_

Re: Hypothetical indexes using BRIN broken since pg10

2019-06-27 Thread Alvaro Herrera
On 2019-Jun-27, Tom Lane wrote: > FWIW, the proposed patch doesn't seem to me like it adds much more > BRIN-specific knowledge to brincostestimate than is there already. It's this calculation that threw me off: statsData.revmapNumPages = (indexRanges / REVMAP_PAGE_MAXITEMS) + 1; ISTM that

Re: Hypothetical indexes using BRIN broken since pg10

2019-06-27 Thread Tom Lane
Alvaro Herrera writes: > On 2019-Jun-27, Tom Lane wrote: >> FWIW, the proposed patch doesn't seem to me like it adds much more >> BRIN-specific knowledge to brincostestimate than is there already. > It's this calculation that threw me off: > statsData.revmapNumPages = (indexRanges / REVMAP_

Re: Hypothetical indexes using BRIN broken since pg10

2019-06-27 Thread Alvaro Herrera
On 2019-Jun-27, Tom Lane wrote: > Alvaro Herrera writes: > > On 2019-Jun-27, Tom Lane wrote: > >> FWIW, the proposed patch doesn't seem to me like it adds much more > >> BRIN-specific knowledge to brincostestimate than is there already. > > > It's this calculation that threw me off: > > stat

Re: Hypothetical indexes using BRIN broken since pg10

2019-06-27 Thread Tom Lane
Alvaro Herrera writes: > On 2019-Jun-27, Tom Lane wrote: >> Um ... it's accounting for revmap pages already (which is why it needs >> this field set), so hasn't that ship sailed? > Yes, but does it need to know how many items there are in a revmap page? Dunno, I just can't get excited about expo

Re: Hypothetical indexes using BRIN broken since pg10

2019-06-27 Thread Alvaro Herrera
On 2019-Jun-27, Tom Lane wrote: > Dunno, I just can't get excited about exposing REVMAP_PAGE_MAXITEMS. > Especially not since we seem to agree on the long-term solution here, > and what you're suggesting to Julien doesn't particularly fit into > that long-term solution. Well, it was brin_page.h,

Re: Missing hook for UPPERREL_PARTIAL_GROUP_AGG rels?

2019-06-27 Thread Erik Nordström
Thanks for the quick response. I do think this might be a separate issue because in the set_rel_pathlist_hook case, that hook is actually called at one point. In this case there is simply no place in the PostgreSQL code where a call is made to create_upper_paths_hook for the UPPERREL_PARTIAL_GROUP_

Re: [HACKERS] Regression tests vs existing users in an installation

2019-06-27 Thread Tom Lane
Robert Haas writes: > On Tue, Jun 25, 2019 at 11:33 PM Tom Lane wrote: >> Comments? > LGTM. Thanks for looking! > s/must/should/ ? Sure, if you like. Further on the rolenames test mess: I started to work on removing that script's creation of out-of-spec user names, but my heart just sank to

Re: GiST VACUUM

2019-06-27 Thread Heikki Linnakangas
On 27/06/2019 20:15, Andrey Borodin wrote: But I have stupid question again, about this code: https://github.com/x4m/postgres_g/commit/096d5586537d29ff316ca8ce074bbe1b325879ee#diff-754126824470cb8e68fd5e32af6d3bcaR417 nextFullXid = ReadNextFullTransactionId();

Re: [HACKERS] Regression tests vs existing users in an installation

2019-06-27 Thread Alvaro Herrera
On 2019-Jun-27, Tom Lane wrote: > Further on the rolenames test mess: I started to work on removing > that script's creation of out-of-spec user names, but my heart just > sank to the floor when I noticed that it was also doing stuff like > this: > > ALTER USER ALL SET application_name to 'SLAP';

Re: GiST VACUUM

2019-06-27 Thread Thomas Munro
On Thu, Jun 27, 2019 at 11:38 PM Heikki Linnakangas wrote: > * I moved the logic to extend a 32-bit XID to 64-bits to a new helper > function in varsup.c. I'm a bit uneasy about making this into a generally available function for reuse. I think we should instead come up with a very small number

Re: make \d pg_toast.foo show its indices ; and, \d toast show its main table ; and \d relkind=I show its partitions (and tablespace)

2019-06-27 Thread Justin Pryzby
My previous patch missed a 1-line hunk, so resending. >From 29e4c0b9700b9dee5f6ff2abc442e08e5221eb93 Mon Sep 17 00:00:00 2001 From: Justin Pryzby Date: Tue, 30 Apr 2019 19:05:53 -0500 Subject: [PATCH v3] print table associated with given TOAST table --- src/bin/psql/describe.c | 22 +

Re: An out-of-date comment in nodeIndexonlyscan.c

2019-06-27 Thread Thomas Munro
On Fri, Feb 8, 2019 at 4:58 AM Thomas Munro wrote: > On Thu, May 17, 2018 at 3:49 AM Heikki Linnakangas wrote: > > On 14/05/18 02:15, Thomas Munro wrote: > > > The first sentence of that comment is no longer true as of commit > > > c01262a8 (2013). As for whether it's necessary to predicate-lock

Re: UCT (Re: pgsql: Update time zone data files to tzdata release 2019a.)

2019-06-27 Thread Andrew Gierth
> "Tom" == Tom Lane writes: > Robert Haas writes: >> I'm kind of unsure what to think about this whole debate >> substantively. If Andrew is correct that zone.tab or zone1970.tab is >> a list of time zone names to be preferred over alternatives, then it >> seems like we ought to prefer

Re: An out-of-date comment in nodeIndexonlyscan.c

2019-06-27 Thread Ashwin Agrawal
On Thu, Jun 27, 2019 at 4:33 PM Thomas Munro wrote: > Here's a patch I'd like to commit to fix the comment. We could look > at improving the actual code after > https://commitfest.postgresql.org/23/2169/ is done. > Change looks good to me. > I wonder if it might be possible to avoid page lock

Re: Usage of epoch in txid_current

2019-06-27 Thread Alexander Korotkov
Hi! On Mon, Jun 24, 2019 at 6:27 PM Andres Freund wrote: > On June 24, 2019 8:19:13 AM PDT, Robert Haas wrote: > >On Fri, Jun 21, 2019 at 7:01 PM Alexander Korotkov > > wrote: > >> On Thu, Mar 28, 2019 at 8:30 AM Thomas Munro > >wrote: > >> > Thanks for the reviews! Pushed. > >> > >> Any ideas

Commitfest 2019-07, the first of five* for PostgreSQL 13

2019-06-27 Thread Thomas Munro
Hello hackers, The first Commitfest[1] for the next major release of PostgreSQL begins in a few days, and runs for the month of July. There are 218 patches registered[2] right now, and I'm sure we'll see some more at the last minute. PostgreSQL 13 needs you! I volunteered to be the CF manager f

Re: Usage of epoch in txid_current

2019-06-27 Thread Alexander Korotkov
On Mon, Jun 24, 2019 at 8:43 PM Alvaro Herrera wrote: > > On 2019-Jun-22, Alexander Korotkov wrote: > > > 2. Also introduce FullMultixactId, and apply to MultixactId the > > similar change as #1. > > 3. Change SLRU on-disk format to handle 64-bit xids and multixacts. > > In particular make SLRU pa

Re: SQL/JSON path issues/questions

2019-06-27 Thread Alexander Korotkov
On Thu, Jun 27, 2019 at 4:57 PM Thom Brown wrote: > On Wed, 19 Jun 2019 at 20:04, Alexander Korotkov > wrote: > > On Wed, Jun 19, 2019 at 7:07 PM Thom Brown wrote: > > > On Thu, 13 Jun 2019 at 14:59, Thom Brown wrote: > > > Also, this example doesn't work: > > > > > > '$.track ? (@.segments[*]

Re: SQL/JSON path issues/questions

2019-06-27 Thread Alexander Korotkov
On Tue, Jun 25, 2019 at 6:38 PM Liudmila Mantrova wrote: > Thank you for the catch! Please see the modified version of patch 0004 > attached. I tried to review and revise the part related to filters, but I failed because I don't understand the notions used in the documentation. What is the diffe

Re: Obsolete comment in commands/analyze.c

2019-06-27 Thread Etsuro Fujita
On Fri, Jun 28, 2019 at 1:02 AM Tomas Vondra wrote: > On Thu, Jun 27, 2019 at 01:53:52PM +0200, Daniel Gustafsson wrote: > >> On 27 Jun 2019, at 13:05, Etsuro Fujita wrote: > > > >> since the citation was also moved to > >> utils/misc/sampling.c, I think the "see full citation below" part > >> sh

Re: SQL/JSON path issues/questions

2019-06-27 Thread Alvaro Herrera
On 2019-Jun-28, Alexander Korotkov wrote: > On Tue, Jun 25, 2019 at 6:38 PM Liudmila Mantrova > wrote: > > Thank you for the catch! Please see the modified version of patch 0004 > > attached. > > I tried to review and revise the part related to filters, but I failed > because I don't understand

Re: An out-of-date comment in nodeIndexonlyscan.c

2019-06-27 Thread Thomas Munro
On Fri, Jun 28, 2019 at 12:55 PM Ashwin Agrawal wrote: > Change looks good to me. Pushed, thanks. -- Thomas Munro https://enterprisedb.com

Re: Hypothetical indexes using BRIN broken since pg10

2019-06-27 Thread Julien Rouhaud
On Thu, Jun 27, 2019 at 10:09 PM Alvaro Herrera wrote: > > On 2019-Jun-27, Tom Lane wrote: > > > Dunno, I just can't get excited about exposing REVMAP_PAGE_MAXITEMS. > > Especially not since we seem to agree on the long-term solution here, > > and what you're suggesting to Julien doesn't particula

Re: SQL/JSON path issues/questions

2019-06-27 Thread Oleg Bartunov
On Fri, Jun 28, 2019 at 8:10 AM Alvaro Herrera wrote: > > On 2019-Jun-28, Alexander Korotkov wrote: > > > On Tue, Jun 25, 2019 at 6:38 PM Liudmila Mantrova > > wrote: > > > Thank you for the catch! Please see the modified version of patch 0004 > > > attached. > > > > I tried to review and revise

Re: catalog files simplification

2019-06-27 Thread Peter Eisentraut
On 2019-06-12 15:34, Tom Lane wrote: > A bigger objection might be that this would leave us with no obvious- > to-the-untrained-eye declaration point for either the struct name or > the two typedef names. That might make tools like ctags sad. Perhaps > it's not really any worse than today, but it

Re: MSVC Build support with visual studio 2019

2019-06-27 Thread Michael Paquier
On Wed, Jun 26, 2019 at 10:29:05PM +1000, Haribabu Kommi wrote: > Thanks for the review. Yes, that patch applies till 9.5, it is my mistake > in naming the patch. I have been able to finally set up an environment with VS 2019 (as usual this stuff needs time, anyway..), and I can confirm that the p

RE: [patch]socket_timeout in interfaces/libpq

2019-06-27 Thread nagaura.ryo...@fujitsu.com
Hi, Michael-san. > From: Michael Paquier > On Wed, Jun 26, 2019 at 04:13:36AM +, nagaura.ryo...@fujitsu.com wrote: > > I don't think that the rest one of my proposals has been rejected > > completely, so I want to restart discussion. > I recall on the matter that there was consensus that nobo

Re: FETCH FIRST clause PERCENT option

2019-06-27 Thread Surafel Temesgen
Hello, The attached patch include the fix for all the comment given regards Surafel On Mon, Apr 1, 2019 at 12:34 AM Andres Freund wrote: > Hi, > > On 2019-03-29 12:04:50 +0300, Surafel Temesgen wrote: > > > + if (node->limitOption == PERCENTAGE) > > +

Extra quote_all_identifiers in _dumpOptions

2019-06-27 Thread Arthur Zakirov
Hello hackers, While working on pg_dump I noticed the extra quote_all_identifiers in _dumpOptions struct. I attached the patch. It came from refactoring by 0eea8047bf. There is also a discussion: https://www.postgresql.org/message-id/CACw0%2B13ZUcXbj9GKJNGZTkym1SXhwRu28nxHoJMoX5Qwmbg_%2Bw%40ma

Re: GiST "choose subtree" support function to inline penalty

2019-06-27 Thread Komяpa
On Thu, Jun 27, 2019 at 6:00 AM Tom Lane wrote: > =?UTF-8?Q?Darafei_=22Kom=D1=8Fpa=22_Praliaskouski?= > writes: > > I'm looking at PostGIS geometry GiST index build times and try to > optimize > > withing the current GiST framework. The function that shows a lot on my > > flame graphs is penalty

Obsolete comment in commands/analyze.c

2019-06-27 Thread Etsuro Fujita
Hi, I think commit 83e176ec1, which moved block sampling functions to utils/misc/sampling.c, forgot to update this comment in commands/analyze.c: "This algorithm is from Jeff Vitter's paper (see full citation below)"; since the citation was also moved to utils/misc/sampling.c, I think the "see ful

Re: mcvstats serialization code is still shy of a load

2019-06-27 Thread Tomas Vondra
On Thu, Jun 27, 2019 at 12:04:30AM -0400, Tom Lane wrote: Tomas Vondra writes: OK. Attached is a patch ditching the alignment in serialized data. I've ditched the macros to access parts of serialized data, and everything gets copied. I lack energy to actually read this patch right now, and I

Re: GiST VACUUM

2019-06-27 Thread Heikki Linnakangas
On 26/06/2019 06:07, Thomas Munro wrote: On Wed, Jun 26, 2019 at 2:33 PM Michael Paquier wrote: On Tue, Jun 25, 2019 at 02:38:43PM +0500, Andrey Borodin wrote: I feel a little uncomfortable with number 0x7fff right in code. PG_INT32_MAX... MaxTransactionId / 2? Yeah, that makes sense

Use relative rpath if possible

2019-06-27 Thread Peter Eisentraut
On several popular operating systems, we can use relative rpaths, using the $ORIGIN placeholder, so that the resulting installation is relocatable. Then we also don't need to set LD_LIBRARY_PATH during make check. This implementation will use a relative rpath if bindir and libdir are under the sa

Re: Obsolete comment in commands/analyze.c

2019-06-27 Thread Daniel Gustafsson
> On 27 Jun 2019, at 13:05, Etsuro Fujita wrote: > since the citation was also moved to > utils/misc/sampling.c, I think the "see full citation below" part > should be updated accordingly. Attached is a patch for that. Agreed, nice catch! cheers ./daniel

Re: GiST VACUUM

2019-06-27 Thread Andrey Borodin
> 27 июня 2019 г., в 16:38, Heikki Linnakangas написал(а): > > I haven't done any testing on this yet. Andrey, would you happen to have an > environment ready to test this? Thanks! I will do some testing this evening (UTC+5). But I'm not sure I can reliably test wraparound of xids... I wil

Re: PG 11 JIT deform failure

2019-06-27 Thread didier
Hi, I searched the mailing list but found nothing. Any reason why TupleDescAttr is a macro and not a static inline? Rather than adding an Assert attached POC replace TupleDescAttr macro by a static inline function with AssertArg. It: - Factorize Assert. - Trigger an Assert in JIT_deform if natts

Re: SQL/JSON path issues/questions

2019-06-27 Thread Thom Brown
On Wed, 19 Jun 2019 at 20:04, Alexander Korotkov wrote: > > On Wed, Jun 19, 2019 at 7:07 PM Thom Brown wrote: > > On Thu, 13 Jun 2019 at 14:59, Thom Brown wrote: > > > > > > Hi, > > > > > > I've been reading through the documentation regarding jsonpath and > > > jsonb_path_query etc., and I have

OpenSSL specific value under USE_SSL instead of USE_OPENSSL

2019-06-27 Thread Daniel Gustafsson
The ssl_ciphers GUC for configuring cipher suites sets the default value based on USE_SSL but it’s actually an OpenSSL specific value. As with other such changes, it works fine now but will become an issue should we support other TLS backends. Attached patch fixes this. cheers ./daniel opens

pg_receivewal documentation

2019-06-27 Thread Jesper Pedersen
Hi, Here is a patch for the pg_receivewal documentation to highlight that WAL isn't acknowledged to be applied. I'll add a CF entry for it. Best regards, Jesper >From 138e09c74db5ea08fd03b4e77853e2ca01742512 Mon Sep 17 00:00:00 2001 From: jesperpedersen Date: Thu, 27 Jun 2019 09:54:44 -0400

Re: [HACKERS] Restricting maximum keep segments by repslots

2019-06-27 Thread Jehan-Guillaume de Rorthais
Hi all, Being interested by this feature, I did a patch review. This features adds the GUC "max_slot_wal_keep_size". This is the maximum amount of WAL that can be kept in "pg_wal" by active slots. If the amount of WAL is superior to this limit, the slot is deactivated and its status (new filed i

Missing PG 12 stable branch

2019-06-27 Thread Bruce Momjian
Does anyone know why there is no PG 12 stable branch in our git tree? $ git branch -l REL9_4_STABLE REL9_5_STABLE REL9_6_STABLE REL_10_STABLE REL_11_STABLE master They exist for earlier releases. Is this a problem? -- Bruce

Re: Missing PG 12 stable branch

2019-06-27 Thread Bruce Momjian
On Thu, Jun 27, 2019 at 10:28:40AM -0400, Bruce Momjian wrote: > Does anyone know why there is no PG 12 stable branch in our git tree? > > $ git branch -l > REL9_4_STABLE > REL9_5_STABLE > REL9_6_STABLE > REL_10_STABLE > REL_11_STABLE > master

Re: Missing PG 12 stable branch

2019-06-27 Thread Tom Lane
Bruce Momjian writes: > Does anyone know why there is no PG 12 stable branch in our git tree? For the record, I'm intending to make the branch as soon as the July CF starts (i.e., first thing Monday). regards, tom lane

Re: Missing PG 12 stable branch

2019-06-27 Thread Tomas Vondra
On Thu, Jun 27, 2019 at 10:28:40AM -0400, Bruce Momjian wrote: Does anyone know why there is no PG 12 stable branch in our git tree? $ git branch -l REL9_4_STABLE REL9_5_STABLE REL9_6_STABLE REL_10_STABLE REL_11_STABLE master T

Re: Obsolete comment in commands/analyze.c

2019-06-27 Thread Tomas Vondra
On Thu, Jun 27, 2019 at 01:53:52PM +0200, Daniel Gustafsson wrote: On 27 Jun 2019, at 13:05, Etsuro Fujita wrote: since the citation was also moved to utils/misc/sampling.c, I think the "see full citation below" part should be updated accordingly. Attached is a patch for that. Agreed, nice

Re: Fix doc bug in logical replication.

2019-06-27 Thread Tomas Vondra
On Sun, Jun 23, 2019 at 10:26:47PM -0400, Robert Treat wrote: On Sun, Jun 23, 2019 at 1:25 PM Peter Eisentraut wrote: On 2019-04-12 19:52, Robert Treat wrote: > It is clear to me that the docs are wrong, but I don't see anything > inherently incorrect about the code itself. Do you have suggest

Re: UCT (Re: pgsql: Update time zone data files to tzdata release 2019a.)

2019-06-27 Thread Tom Lane
Andrew Gierth writes: > "Tom" == Tom Lane writes: > Tom> In general, the point I'm trying to make is that our policy should > Tom> be "Ties are broken arbitrarily, and if you don't like the choice > Tom> that initdb makes, here's how to fix it". > Yes, you've repeated that point at some lengt

Re: GiST VACUUM

2019-06-27 Thread Andrey Borodin
> 27 июня 2019 г., в 16:38, Heikki Linnakangas написал(а): > > I haven't done any testing on this yet. Andrey, would you happen to have an > environment ready to test this? Patches do not deadlock and delete pages on "rescan test" - setup that we used to detect "left jumps" in during develo

Re: UCT (Re: pgsql: Update time zone data files to tzdata release 2019a.)

2019-06-27 Thread Robert Haas
On Tue, Jun 25, 2019 at 8:18 PM Tom Lane wrote: > As long as we have a trivial and obviously apolitical rule like > alphabetical order, I think we can skate over such things; but the minute > we have any sort of human choices involved there, we're going to be > getting politically driven requests

Re: [HACKERS] Regression tests vs existing users in an installation

2019-06-27 Thread Robert Haas
On Tue, Jun 25, 2019 at 11:33 PM Tom Lane wrote: > Comments? LGTM. s/must/should/ ? -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company

Re: Fix doc bug in logical replication.

2019-06-27 Thread Dave Cramer
On Thu, 27 Jun 2019 at 12:50, Tomas Vondra wrote: > On Sun, Jun 23, 2019 at 10:26:47PM -0400, Robert Treat wrote: > >On Sun, Jun 23, 2019 at 1:25 PM Peter Eisentraut > > wrote: > >> > >> On 2019-04-12 19:52, Robert Treat wrote: > >> > It is clear to me that the docs are wrong, but I don't see any

Re: UCT (Re: pgsql: Update time zone data files to tzdata release 2019a.)

2019-06-27 Thread Tom Lane
Robert Haas writes: > I'm kind of unsure what to think about this whole debate > substantively. If Andrew is correct that zone.tab or zone1970.tab is a > list of time zone names to be preferred over alternatives, then it > seems like we ought to prefer them. It's not really clear to me that the I

Hypothetical indexes using BRIN broken since pg10

2019-06-27 Thread Julien Rouhaud
Hello, I just realized that 7e534adcdc7 broke support for hypothetical indexes using BRIN am. Attached patch fix the issue. There's no interface to provide the hypothetical pagesPerRange value, so I used the default one, and used simple estimates. I'll add this patch to the next commitfest. dif

Re: POC: Cleaning up orphaned files using undo logs

2019-06-27 Thread Robert Haas
On Tue, Jun 25, 2019 at 4:00 PM Amit Kapila wrote: > [ new patches ] I happened to open up 0001 from this series, which is from Thomas, and I do not think that the pg_buffercache changes are correct. The idea here is that the customer might install version 1.3 or any prior version on an old rele

Re: Hypothetical indexes using BRIN broken since pg10

2019-06-27 Thread Alvaro Herrera
Hi, thanks for the patch. On 2019-Jun-27, Julien Rouhaud wrote: > I just realized that 7e534adcdc7 broke support for hypothetical > indexes using BRIN am. Attached patch fix the issue. > > There's no interface to provide the hypothetical pagesPerRange value, > so I used the default one, and use

Re: Fix doc bug in logical replication.

2019-06-27 Thread Tomas Vondra
On Thu, Jun 27, 2019 at 01:46:47PM -0400, Dave Cramer wrote: On Thu, 27 Jun 2019 at 12:50, Tomas Vondra wrote: On Sun, Jun 23, 2019 at 10:26:47PM -0400, Robert Treat wrote: >On Sun, Jun 23, 2019 at 1:25 PM Peter Eisentraut > wrote: >> >> On 2019-04-12 19:52, Robert Treat wrote: >> > It is clea

Re: Hypothetical indexes using BRIN broken since pg10

2019-06-27 Thread Julien Rouhaud
On Thu, Jun 27, 2019 at 8:14 PM Alvaro Herrera wrote: > > Hi, thanks for the patch. Thanks for looking at it! > On 2019-Jun-27, Julien Rouhaud wrote: > > > I just realized that 7e534adcdc7 broke support for hypothetical > > indexes using BRIN am. Attached patch fix the issue. > > > > There's no

Re: pglz performance

2019-06-27 Thread Andrey Borodin
> 13 мая 2019 г., в 12:14, Michael Paquier написал(а): > > Decompression can matter a lot for mostly-read workloads and > compression can become a bottleneck for heavy-insert loads, so > improving compression or decompression should be two separate > problems, not two problems linked. Any impr

Re: Fix doc bug in logical replication.

2019-06-27 Thread Dave Cramer
On Thu, 27 Jun 2019 at 14:20, Tomas Vondra wrote: > On Thu, Jun 27, 2019 at 01:46:47PM -0400, Dave Cramer wrote: > >On Thu, 27 Jun 2019 at 12:50, Tomas Vondra > >wrote: > > > >> On Sun, Jun 23, 2019 at 10:26:47PM -0400, Robert Treat wrote: > >> >On Sun, Jun 23, 2019 at 1:25 PM Peter Eisentraut >

Re: Hypothetical indexes using BRIN broken since pg10

2019-06-27 Thread Alvaro Herrera
On 2019-Jun-27, Julien Rouhaud wrote: > On Thu, Jun 27, 2019 at 8:14 PM Alvaro Herrera > wrote: > > I think it would look nicer to have a routine parallel to brinGetStats() > > (brinGetStatsHypothetical?), instead of polluting selfuncs.c with these > > gory details. > > I'm not opposed to it,

  1   2   >