Re: Problems with "pg.dropped" column after upgrade 9.5 to 9.6

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Michael Paquier <michael(dot)paquier(at)gmail(dot)com>
Cc: Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>, Pavel Hanák <hanak(at)is-it(dot)eu>, "pgsql-bugs(at)postgresql(dot)org" <pgsql-bugs(at)postgresql(dot)org>
Subject: Re: Problems with "pg.dropped" column after upgrade 9.5 to 9.6
Date: 2016-11-02 13:17:43
Message-ID: 9842.1478092663@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs

Michael Paquier <michael(dot)paquier(at)gmail(dot)com> writes:
> And one bisect later:
> commit: 3fc6e2d7f5b652b417fa6937c34de2438d60fa9f
> author: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
> date: Mon, 7 Mar 2016 15:58:22 -0500
> Make the upper part of the planner work by generating and comparing Paths

Yeah. The reimplementation I did in createplan.c causes the planner to
generate a targetlist that seems valid in practice, but doesn't pass the
rather simplistic test in ExecCheckPlanOutput. (The latter is expecting
a simple null Const corresponding to a dropped column, but what it's
getting is a Var that references a null Const one plan level down.)

Not sure whether it's reasonable to make the planner jump through extra
hoops here, or complicate the null-for-dropped-cols test in
ExecCheckPlanOutput, or just delete that as not really needed.

regards, tom lane

In response to

Browse pgsql-bugs by date

  From Date Subject
Next Message Tom Lane 2016-11-02 13:29:12 Re: Compilation of timezone source with zic fails (on mountpoint)
Previous Message Sandeep Thakkar 2016-11-02 12:41:17 Re: Compilation of timezone source with zic fails (on mountpoint)