Re: pgsql: New files for MERGE

Hi,

On 2018-04-03 08:32:45 -0700, Andres Freund wrote:
> Hi,
> 
> On 2018-04-03 09:24:12 +0000, Simon Riggs wrote:
> > New files for MERGE
> > src/backend/executor/nodeMerge.c                   |  575 +++++++
> > src/backend/parser/parse_merge.c                   |  660 ++++++++
> > src/include/executor/nodeMerge.h                   |   22 +
> > src/include/parser/parse_merge.h                   |   19 +
> 
> Getting a bit grumpy here.  So you pushed this, without responding in
> any way to the objections I made in
> http://archives.postgresql.org/message-id/20180403021800.b5nsgiclzanobiup%40alap3.anarazel.de
> and did it in a manner that doesn't even compile?

This needs at the very least a response to the issues pointed out in the
referenced email that you chose to ignore without any sort of comment.

Greetings,

Andres Freund