Re: [HACKERS] Unused field of ProjectionPath

2017-08-03 Thread Tom Lane
Antonin Houska 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

[HACKERS] Unused field of ProjectionPath

2017-08-03 Thread Antonin Houska
I've noticed that the dummypp field of ProjectionPath is set but never read. If the only possible change between the path and plan creation time is that the projection path and the subpath targetlists become different, then dummypp could be used this way: diff --git