Re: Unused field of ProjectionPath

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Antonin Houska <ah(at)cybertec(dot)at>
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: Unused field of ProjectionPath
Date: 2017-08-03 14:04:43
Message-ID: 6217.1501769083@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Antonin Houska <ah(at)cybertec(dot)at> writes:
> I've noticed that the dummypp field of ProjectionPath is set but never read.

I do not think removing that field is a good idea, even if it's not used
in the current form of the logic. It's useful for debugging purposes at
least. See original commit comment at

https://git.postgresql.org/gitweb/?p=postgresql.git;a=commitdiff;h=8b9d323cb9810109e3e5aab1ead427cbbb7aa77e

regards, tom lane

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message David G. Johnston 2017-08-03 14:08:17 Re: reload-through-the-top-parent switch the partition table
Previous Message Oliver Ford 2017-08-03 13:25:56 Add Roman numeral conversion to to_number