Re: Typo in recent commit

2017-12-09 Thread Michael Paquier
On Sun, Dec 10, 2017 at 11:51 AM, Robins Tharakan wrote: > However, although it was backpatched correctly, looks like the fix on master > missed out identity.out related fix. Don't worry. This has been addressed already on HEAD and REL_10_STABLE. See ce1468d. -- Michael

Re: Typo in recent commit

2017-12-09 Thread Robins Tharakan
On 9 December 2017 at 16:11, Magnus Hagander wrote: > ​​ > > Thanks, fixed for the report. > > Thanks Magnus. However, although it was backpatched correctly, looks like the fix on master missed out identity.out related fix.​ Ref:

Rethinking MemoryContext creation

2017-12-09 Thread Tom Lane
While fooling around with a different problem, I got annoyed at how slow MemoryContext creation and deletion is. A big chunk of that seems to be the palloc/pfree cycle needed to allocate the context header block in TopMemoryContext. After thinking about it for a bit, I decided that it'd be

Re: proposal: alternative psql commands quit and exit

2017-12-09 Thread Tom Lane
Peter Eisentraut writes: > On 12/8/17 10:19, Tom Lane wrote: >> Meh. I never thought that the "help" business was particularly well >> thought out, and this proposal isn't better. > The perhaps the fix is to revert the "help" thing? No, my problem with it is

Re: proposal: alternative psql commands quit and exit

2017-12-09 Thread hvjunk
This one I’ll support to have those messages brought back for psql (especially when it’s connected to a pty), as then the SO question would fade away into obscurity (At least for those users that actually read :D ) > On 09 Des. 2017, at 11:45 nm., Peter Eisentraut >

Re: Jsonb transform for pl/python

2017-12-09 Thread Peter Eisentraut
On 12/6/17 06:40, Anthony Bykov wrote: > Hello, > fixed the issues: > 1. Rising errors when invalid object being transformed. > 2. We don't rise the exception when transforming range(3) only in > python 2. In third one it is an error. > > Please, find the 4-th version of the patch in attachments

Re: proposal: alternative psql commands quit and exit

2017-12-09 Thread Peter Eisentraut
On 12/8/17 10:19, Tom Lane wrote: > Meh. I never thought that the "help" business was particularly well > thought out, and this proposal isn't better. The irony is that before the "help" thing was put in, the psql startup message said Type: \copyright for distribution terms \h for help

Re: proposal: alternative psql commands quit and exit

2017-12-09 Thread Peter Eisentraut
On 12/8/17 14:08, hvjunk wrote: > Sounds like the VI exit issue? > https://stackoverflow.com/questions/11828270/how-to-exit-the-vim-editor > > Having “made that ‘mistake’” once, by lots of users, does that imply it > is a constant problem for those users? Yeah, I'm not sure I buy the way we are

Re: [HACKERS] Parallel tuplesort (for parallel B-Tree index creation)

2017-12-09 Thread Peter Geoghegan
On Thu, Dec 7, 2017 at 11:28 PM, Rushabh Lathia wrote: > I thought parallel_leader_participation is generic GUC which get effect > for all parallel operation. isn't it? On that understanding I just update > the > documentation of parallel_leader_participation into

SIGPIPE in TAP tests

2017-12-09 Thread Noah Misch
Two buildfarm runs[1][2] from the last 90 days have failed in src/test/authentication, like this: t/001_password.pl .. Failed 3/8 subtests t/002_saslprep.pl .. ok 1815 ms ( 0.00 usr 0.00 sys + 0.89 cusr 0.26 csys = 1.15 CPU) Test Summary Report ---

Re: pl/perl extension fails on Windows

2017-12-09 Thread Noah Misch
On Wed, Nov 29, 2017 at 11:45:35PM -0500, Tom Lane wrote: > Noah Misch writes: > > On Wed, Nov 29, 2017 at 11:34:56PM -0500, Tom Lane wrote: > >> I don't really have an opinion about the relative merits of these changes, > >> but why do anything? The existing solution has the

Re: Building PL/Perl with ActiveState Perl 5.22 and MSVC

2017-12-09 Thread Noah Misch
On Mon, Nov 13, 2017 at 07:41:42AM +, Noah Misch wrote: > On Sat, Mar 26, 2016 at 03:43:21PM +0300, Victor Wagner wrote: > > 1. ActiveState Perl doesn't ship MSVC-build import library perl522.lib > > for their perl522.dll. Instead they ship MINGW-build library > > libperl522.a. > > 2. There

Re: plpgsql fails to reinitialize record variables at block re-entry

2017-12-09 Thread Tom Lane
Pavel Stehule writes: > 2017-12-09 7:24 GMT+01:00 Tom Lane : >> Surely that ought to have failed at the i=2 iteration. There is >> code in plpgsql's exec_stmt_block() that tries to reinitialize >> PLPGSQL_DTYPE_REC variables, but it's never reached

Re: proposal: alternative psql commands quit and exit

2017-12-09 Thread David Fetter
On Fri, Dec 08, 2017 at 02:12:06PM -0500, Robert Haas wrote: > On Fri, Dec 8, 2017 at 10:28 AM, Szymon Lipiński wrote: > > Well, if I have a new program I usually read some documentation. It really > > helps people to exit vim as well :) > > Look, I love vim and use it

Re: proposal: alternative psql commands quit and exit

2017-12-09 Thread Daniel Vérité
Everaldo Canuto wrote: > > > > postgres=> select 2+2 > >2-> help > >3-> quit > >4-> exit > > > I like it! Maybe another proposal to change PROMPT2? Thanks. Yes, I think it should be a different proposal. On closer inspection, mysql doesn't make a real effort to

Re: proposal: alternative psql commands quit and exit

2017-12-09 Thread Scott Mead
On Sat, Dec 9, 2017 at 1:27 AM, Rok Kralj wrote: > Most popular questions on every topic are the simplest ones. Imagine a > parallel universe where this patch is contained in the codebase - I argue > this question about exiting the repl still is the top one, albeit with less

Re: proposal: alternative psql commands quit and exit

2017-12-09 Thread Everaldo Canuto
On Sat, Dec 9, 2017 at 11:36 AM, Daniel Vérité" wrote: > > postgres=> select 2+2 >2-> help >3-> quit >4-> exit I like it! Maybe another proposal to change PROMPT2?

Re: pgsql: Prohibit identity columns on typed tables and partitions

2017-12-09 Thread Magnus Hagander
On Sat, Dec 9, 2017 at 10:45 PM, Peter Eisentraut < peter.eisentr...@2ndquadrant.com> wrote: > On 12/9/17 05:47, Magnus Hagander wrote: > > > > > > On Sat, Dec 9, 2017 at 8:43 AM, Michael Paquier > > > wrote: > > > > On Sat, Dec 9,

Re: pgsql: Prohibit identity columns on typed tables and partitions

2017-12-09 Thread Peter Eisentraut
On 12/9/17 05:47, Magnus Hagander wrote: > > > On Sat, Dec 9, 2017 at 8:43 AM, Michael Paquier > > wrote: > > On Sat, Dec 9, 2017 at 2:26 AM, Peter Eisentraut > wrote: > >

Re: Speeding up pg_upgrade

2017-12-09 Thread Stephen Frost
Bruce, * Bruce Momjian (br...@momjian.us) wrote: > On Fri, Dec 8, 2017 at 12:26:55PM -0500, Stephen Frost wrote: > > * Bruce Momjian (br...@momjian.us) wrote: > > > I think the big problem with two-stage pg_upgrade is that the user steps > > > are more complex, so what percentage of users are

Re: explain analyze output with parallel workers - question about meaning of information for explain.depesz.com

2017-12-09 Thread Amit Kapila
On Sat, Dec 9, 2017 at 1:24 AM, Robert Haas wrote: > On Fri, Dec 8, 2017 at 5:11 AM, Amit Kapila wrote: >>> Okay, I have adjusted the patch accordingly. I have also added a >>> regression test which should produce the same result across different