Re: [HACKERS] md.c is feeling much better now, thank you

Tatsuo Ishii <t-ishii(at)sra(dot)co(dot)jp> writes:
>> Hmm. It seems a more straightforward solution would be to alter
>> pg_parse_and_plan so that the parser isn't even called if we have
>> already failed the current transaction; that is, the "queries ignored"
>> test should occur sooner. I'm rather surprised to realize that
>> we do run the parser in this situation...

> No. we have to run the parser so that we could accept "end".

Ah, very good point. I stand corrected.

>>>> Until that I propose following solution. It looks
>>>> simple, safe and would be neccessary anyway (I don't know why that
>>>> check had not been implemented). See included patches.
>>
>> This looks like it might be a good change, but I'm not quite as sure
>> as you are that it won't have any bad effects. Have you tested it?
>
> At least initdb and the regression test runs fine for me...

Same here. I have committed it into current, but not REL6_5.

regards, tom lane