RE: SQL statement PREPARE does not work in ECPG

2019-05-30 Thread Matsumura, Ryo
Meskes-san > This looks good to me. It passes all my tests, too. > > There were two minor issues, the regression test did not run and gcc > complained about the indentation in ECPGprepare(). Both I easily fixed. Thank you so much ! > > (2) > > I found some bugs (two types). I didn't fix them an

Re: Minimal logical decoding on standbys

2019-05-30 Thread Amit Khandekar
On Thu, 30 May 2019 at 20:13, Andres Freund wrote: > > Hi, > > On 2019-05-30 19:46:26 +0530, Amit Khandekar wrote: > > @@ -1042,7 +1042,8 @@ ReplicationSlotReserveWal(void) > > if (!RecoveryInProgress() && SlotIsLogical(slot)) > > { > > > > } > > else > > { > > - restart_lsn = GetR

Re: New committer: David Rowley

2019-05-30 Thread Alexander Korotkov
On Thu, May 30, 2019 at 6:39 PM Magnus Hagander wrote: > For those of you that have not read the minutes from the developer meeting > ahead of pgcon (can be found at > https://wiki.postgresql.org/wiki/PgCon_2019_Developer_Meeting), we'd like to > announce here as well that David Rowley has join

Re: Fix inconsistencies for v12

2019-05-30 Thread Amit Langote
On 2019/05/30 18:51, Amit Kapila wrote: > On Wed, May 29, 2019 at 6:12 AM Amit Langote wrote: >> On 2019/05/28 20:26, Amit Kapila wrote: >>> Can we ensure some way that only FDW's rely on it not any other part >>> of the code? >> >> Hmm, I can't think of any way of doing than other than manual insp

Re: New committer: David Rowley

2019-05-30 Thread Amit Langote
On 2019/05/31 0:39, Magnus Hagander wrote: > Hi! > > For those of you that have not read the minutes from the developer meeting > ahead of pgcon (can be found at > https://wiki.postgresql.org/wiki/PgCon_2019_Developer_Meeting), we'd like > to announce here as well that David Rowley has joined the

Re: How to know referenced sub-fields of a composite type?

2019-05-30 Thread Kohei KaiGai
2019/05/30 16:33、Amit Langote のメール: >> On 2019/05/29 15:50, Kohei KaiGai wrote: >> 2019年5月29日(水) 13:26 Amit Langote : It means we can skip to load the sub-fields unreferenced, if query-planner can handle referenced and unreferenced sub-fields correctly. On the other hands, it l

Re: incorrect xlog.c coverage report

2019-05-30 Thread Michael Paquier
On Wed, May 29, 2019 at 12:09:08PM -0400, Alvaro Herrera wrote: > Are there objections to doing that now on the master branch? Adding the flush call just on HEAD is fine for me. Not sure that there is an actual reason to back-patch that. -- Michael signature.asc Description: PGP signature

Re: cpluspluscheck vs vpath

2019-05-30 Thread Alvaro Herrera
On 2019-May-30, Andres Freund wrote: > On 2019-05-30 15:02:44 -0700, Andres Freund wrote: > > > Seems we could round the edges a good bit further than what's done in > > the attached (argument checking, for example, but also using the C++ > > compiler from configure). But I think this would alread

Re: cpluspluscheck vs vpath

2019-05-30 Thread Andres Freund
Hi, On 2019-05-30 15:02:44 -0700, Andres Freund wrote: > right now cpluspluscheck doesn't work with vpath builds. That's pretty > annoying, because it does require cloning the git tree into a separate > directory + doing configure there just to run cpluspluscheck. > > Attached is a small patch al

cpluspluscheck vs vpath

2019-05-30 Thread Andres Freund
Hi, right now cpluspluscheck doesn't work with vpath builds. That's pretty annoying, because it does require cloning the git tree into a separate directory + doing configure there just to run cpluspluscheck. Attached is a small patch allowing cpluspluscheck to run from different directories. I ne

Re: Ought to use heap_multi_insert() for pg_attribute/depend insertions?

2019-05-30 Thread Andres Freund
Hi, On 2019-05-22 10:25:14 +0200, Daniel Gustafsson wrote: > It passes make check and some light profiling around regress suites indicates > that it does improve a bit by reducing the somewhat costly calls. Just for the record, here is the profile I did: perf record --call-graph lbr make -s chec

Re: tableam.h fails cpluspluscheck

2019-05-30 Thread Andres Freund
Hi, On 2019-05-30 14:01:00 -0400, Tom Lane wrote: > Using Apple's clang as c++ compiler: > > In file included from /tmp/cpluspluscheck.KejiIw/test.cpp:3: > ./src/include/access/tableam.h:144:16: error: typedef redefinition with > different types ('void (*)(Relation, HeapTuple, Datum *, bool *, b

Re: New committer: David Rowley

2019-05-30 Thread Michael Paquier
On Thu, May 30, 2019 at 11:39:23AM -0400, Magnus Hagander wrote: > Congratulations to David, may the buildfarm be gentle to him, and his first > revert far away! Congrats! -- Michael signature.asc Description: PGP signature

Re: Why does pg_checksums -r not have a long option?

2019-05-30 Thread Michael Paquier
On Mon, May 27, 2019 at 04:22:37PM +0200, Fabien COELHO wrote: > Works for me. Doc build is ok as well. Thanks, committed. -- Michael signature.asc Description: PGP signature

Re: coverage additions

2019-05-30 Thread Michael Paquier
On Thu, May 30, 2019 at 01:52:20PM -0400, Alvaro Herrera wrote: > If there are other obvious improvements to be had, please let me know. > (We have PG_TEST_EXTRA="ssl ldap" currently, do we have any more extra > tests now?) You can add kerberos to this list, to give: PG_TEST_EXTRA='ssl ldap kerber

Re: coverage additions

2019-05-30 Thread Tom Lane
Alvaro Herrera writes: > Apparently, for ecpg you have to do "make checktcp" in order for some of > the tests to run, and "make check-world" doesn't do that. Not sure > what's a good fix for this; do we want to add "make -C > src/interfaces/ecpg/test checktcp" to what "make check-world" does, > o

Re: compiling PL/pgSQL plugin with C++

2019-05-30 Thread Tom Lane
I wrote: > I propose that we change src/tools/pginclude/cpluspluscheck so that > it searches basically everywhere: > -for f in `find src/include src/interfaces/libpq/libpq-fe.h > src/interfaces/libpq/libpq-events.h -name '*.h' -print | \ > +for f in `find src contrib -name '*.h' -print | \ Afte

Re: coverage additions

2019-05-30 Thread Alvaro Herrera
On 2019-May-30, Tom Lane wrote: > Alvaro Herrera writes: > > I just enabled --enabled-llvm on the coverage reporting machine, which > > made src/backend/jit/jit.c go from 60/71 % (line/function wise) to 78/85 % > > ... > > and src/backend/jit/llvm from not appearing at all in the report to > > 7

Re: coverage additions

2019-05-30 Thread Alvaro Herrera
Apparently, for ecpg you have to do "make checktcp" in order for some of the tests to run, and "make check-world" doesn't do that. Not sure what's a good fix for this; do we want to add "make -C src/interfaces/ecpg/test checktcp" to what "make check-world" does, or do we rather what to add checktc

Re: New committer: David Rowley

2019-05-30 Thread Joe Conway
On 5/30/19 11:43 AM, Andres Freund wrote: > Hi, > > On 2019-05-30 11:39:23 -0400, Magnus Hagander wrote: >> For those of you that have not read the minutes from the developer meeting >> ahead of pgcon (can be found at >> https://wiki.postgresql.org/wiki/PgCon_2019_Developer_Meeting), we'd like >>

Re: New committer: David Rowley

2019-05-30 Thread Larry Rosenman
On 05/30/2019 10:39 am, Magnus Hagander wrote: > Hi! > > For those of you that have not read the minutes from the developer meeting > ahead of pgcon (can be found at > https://wiki.postgresql.org/wiki/PgCon_2019_Developer_Meeting), we'd like to > announce here as well that David Rowley has jo

Re: New committer: David Rowley

2019-05-30 Thread Amit Kapila
On Thu, May 30, 2019 at 9:09 PM Magnus Hagander wrote: > > Hi! > > For those of you that have not read the minutes from the developer meeting > ahead of pgcon (can be found at > https://wiki.postgresql.org/wiki/PgCon_2019_Developer_Meeting), we'd like to > announce here as well that David Rowle

Re: coverage additions

2019-05-30 Thread Tom Lane
Alvaro Herrera writes: > I just enabled --enabled-llvm on the coverage reporting machine, which > made src/backend/jit/jit.c go from 60/71 % (line/function wise) to 78/85 % ... > and src/backend/jit/llvm from not appearing at all in the report to > 78/94 %. That's a good improvement. > If there

tableam.h fails cpluspluscheck

2019-05-30 Thread Tom Lane
Using Apple's clang as c++ compiler: In file included from /tmp/cpluspluscheck.KejiIw/test.cpp:3: ./src/include/access/tableam.h:144:16: error: typedef redefinition with different types ('void (*)(Relation, HeapTuple, Datum *, bool *, bool, void *)' (aka 'void (*)(RelationData *, HeapTupleData *

coverage additions

2019-05-30 Thread Alvaro Herrera
I just enabled --enabled-llvm on the coverage reporting machine, which made src/backend/jit/jit.c go from 60/71 % (line/function wise) to 78/85 % ... and src/backend/jit/llvm from not appearing at all in the report to 78/94 %. That's a good improvement. If there are other obvious improvements to

Applicant for Google Season of Documentation

2019-05-30 Thread Devansh Gupta
Hi, I, Devansh Gupta, have just completed my sophomore year in B.Tech. in Computer Science and Engineering from International Institute of Information Technology Hyderabad, India and am planning to contribute to the documentation of the *postgreSQL *project. I have already used numPy for many pro

Re: coverage increase for worker_spi

2019-05-30 Thread Tom Lane
Alvaro Herrera writes: > On 2019-May-30, Tom Lane wrote: >> Hm, I don't understand how this works at all: >> >> +PERFORM pg_sleep(CASE WHEN count(*) = 0 THEN 0 ELSE 0.1 >> END) >> +FROM schema1.counted WHERE type = 'delta'; >> +GET DIAG

Re: New committer: David Rowley

2019-05-30 Thread David Fetter
On Thu, May 30, 2019 at 11:39:23AM -0400, Magnus Hagander wrote: > Hi! > > For those of you that have not read the minutes from the developer meeting > ahead of pgcon (can be found at > https://wiki.postgresql.org/wiki/PgCon_2019_Developer_Meeting), we'd like > to announce here as well that David

Re: New committer: David Rowley

2019-05-30 Thread Andres Freund
Hi, On 2019-05-30 11:39:23 -0400, Magnus Hagander wrote: > For those of you that have not read the minutes from the developer meeting > ahead of pgcon (can be found at > https://wiki.postgresql.org/wiki/PgCon_2019_Developer_Meeting), we'd like > to announce here as well that David Rowley has joine

Re: coverage increase for worker_spi

2019-05-30 Thread Alvaro Herrera
On 2019-May-30, Tom Lane wrote: > Alvaro Herrera writes: > > On 2019-May-29, Tom Lane wrote: > >> I'm not opposed to adding a new test case at this point in the cycle, > >> but as written this one seems more or less guaranteed to fail under > >> load. > > > True. Here's a version that should be

Re: coverage increase for worker_spi

2019-05-30 Thread Tom Lane
Alvaro Herrera writes: > On 2019-May-29, Tom Lane wrote: >> I'm not opposed to adding a new test case at this point in the cycle, >> but as written this one seems more or less guaranteed to fail under >> load. > True. Here's a version that should be more resilient. Hm, I don't understand how th

Re: New committer: David Rowley

2019-05-30 Thread David Steele
On 5/30/19 11:39 AM, Magnus Hagander wrote: For those of you that have not read the minutes from the developer meeting ahead of pgcon (can be found at https://wiki.postgresql.org/wiki/PgCon_2019_Developer_Meeting), we'd like to announce here as well that David Rowley has joined the ranks of

Re: error messages in extended statistics

2019-05-30 Thread Alvaro Herrera
On 2019-May-30, Tomas Vondra wrote: > Pushed and backpatched, changing most places to elog(). Thanks :-) -- Álvaro Herrerahttps://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

Re: coverage increase for worker_spi

2019-05-30 Thread Alvaro Herrera
On 2019-May-29, Tom Lane wrote: > Alvaro Herrera writes: > > Tom pointed out that coverage for worker_spi is 0%. For a module that > > only exists to provide coverage, that's pretty stupid. This patch > > increases coverage to 90.9% line-wise and 100% function-wise, which > > seems like a suffi

Re: compiling PL/pgSQL plugin with C++

2019-05-30 Thread Tom Lane
[ redirecting to -hackers ] =?koi8-r?B?9MHSwdPP1yDnxc/Sx8nKIPfJ1MHM2MXXyd4=?= writes: > I'm working on development of some PL/pgSQL plugin. > The smaller part of my code is written on C. > It's a standard extension code for integration with fmgr (_PG_init ...) > But bigger part of the code is wri

Re: New committer: David Rowley

2019-05-30 Thread Marko Tiikkaja
On Thu, May 30, 2019 at 6:39 PM Magnus Hagander wrote: For those of you that have not read the minutes from the developer meeting ahead of pgcon (can be found at https://wiki.postgresql.org/wiki/PgCon_2019_Developer_Meeting), we'd like to announce here as well that David Rowley has joined the rank

New committer: David Rowley

2019-05-30 Thread Magnus Hagander
Hi! For those of you that have not read the minutes from the developer meeting ahead of pgcon (can be found at https://wiki.postgresql.org/wiki/PgCon_2019_Developer_Meeting), we'd like to announce here as well that David Rowley has joined the ranks of PostgreSQL committers. Congratulations to Dav

Re: error messages in extended statistics

2019-05-30 Thread Tomas Vondra
On Wed, May 15, 2019 at 06:35:47PM +0200, Tomas Vondra wrote: On Wed, May 15, 2019 at 12:17:29PM -0400, Alvaro Herrera wrote: On 2019-May-05, Tomas Vondra wrote: OK, so here is a patch, using elog() for all places except for the input function, where we simply report we don't accept those valu

Re: Zedstore - compressed in-core columnar storage

2019-05-30 Thread DEV_OPS
it's really cool and very good progress,  I'm interesting if SIDM/JIT will be supported best wishes TY On 2019/5/23 08:07, Ashwin Agrawal wrote: > We (Heikki, me and Melanie) are continuing to build Zedstore. Wish to > share the recent additions and modifications. Attaching a patch > with th

Re: Server crash due to assertion failure in CheckOpSlotCompatibility()

2019-05-30 Thread Andres Freund
Hi, On 2019-05-30 16:31:39 +0530, Ashutosh Sharma wrote: > Here are some more details on the crash reported in my previous e-mail for > better clarity: I'll look into this once pgcon is over... Thanks for finding! Greetings, Andres Freund

Re: Minimal logical decoding on standbys

2019-05-30 Thread Andres Freund
Hi, On 2019-05-30 19:46:26 +0530, Amit Khandekar wrote: > @@ -1042,7 +1042,8 @@ ReplicationSlotReserveWal(void) > if (!RecoveryInProgress() && SlotIsLogical(slot)) > { > > } > else > { > - restart_lsn = GetRedoRecPtr(); > + restart_lsn = SlotIsLogical(slot) ? > +

Re: [HACKERS] [PATCH] Generic type subscripting

2019-05-30 Thread Dmitry Dolgov
> On Wed, May 29, 2019 at 6:17 PM Pavel Stehule wrote: > > st 29. 5. 2019 v 17:49 odesílatel Dmitry Dolgov <9erthali...@gmail.com> > napsal: >> >> Rebase after pg_indent. Besides, off the list there was a suggestion that >> this >> could be useful to accept more than one data type as a key for s

Re: Minimal logical decoding on standbys

2019-05-30 Thread Amit Khandekar
On Mon, 27 May 2019 at 19:26, Andres Freund wrote: > > On 2019-05-27 17:04:44 +0530, Amit Khandekar wrote: > > On Fri, 24 May 2019 at 21:00, Amit Khandekar wrote: > > > > > > On Fri, 24 May 2019 at 19:26, Amit Khandekar > > > wrote: > > > > Working on the patch now > > > > > > Attached is

Re: Server crash due to assertion failure in CheckOpSlotCompatibility()

2019-05-30 Thread Ashutosh Sharma
Hi All, Here are some more details on the crash reported in my previous e-mail for better clarity: The crash only happens when a *primary key* or *btree index* is created on the test table. For example consider the following two scenarios. *TC1: With PK* create table t1(a int *primary key*, b te

Re: Fix inconsistencies for v12

2019-05-30 Thread Amit Kapila
On Wed, May 29, 2019 at 6:12 AM Amit Langote wrote: > > On 2019/05/28 20:26, Amit Kapila wrote: > > On Tue, May 28, 2019 at 12:29 PM Amit Langote wrote: > >> Seeing that the crash occurs due to postgres_fdw relying on > >> es_result_relation_info being set when initializing a "direct > >> modifica

Re: Fix inconsistencies for v12

2019-05-30 Thread Amit Kapila
On Tue, May 28, 2019 at 10:30 AM Alexander Lakhin wrote: > > 28.05.2019 2:05, Amit Kapila wrote: > > On Mon, May 27, 2019 at 3:59 AM Tom Lane wrote: > >> Amit Kapila writes: > >>> On Sun, May 26, 2019 at 2:20 AM Alexander Lakhin > >>> wrote: > 5. ExecContextForcesOids - not changed, but m

Re: How to know referenced sub-fields of a composite type?

2019-05-30 Thread Amit Langote
On 2019/05/29 15:50, Kohei KaiGai wrote: > 2019年5月29日(水) 13:26 Amit Langote : >>> It means we can skip to load the sub-fields unreferenced, if >>> query-planner can handle >>> referenced and unreferenced sub-fields correctly. >>> On the other hands, it looks to me RelOptInfo or other optimizer >>>

Re: Pinned files at Windows

2019-05-30 Thread Konstantin Knizhnik
On 29.05.2019 22:20, Michael Paquier wrote: On Mon, May 27, 2019 at 05:52:13PM +0300, Konstantin Knizhnik wrote: Postgres is opening file with FILE_SHARE_DELETE  flag which makes it possible to unlink opened file. But unlike Unixes, the file is not actually deleted. You can see it using "dir"