(2018/04/25 11:05), Alvaro Herrera wrote:
Amit Langote wrote:
I think we should remove the words "if not already done" from that
comment because 1) that function is called if the partition wasn't
already initialized and 2) that function assumes that. Attached is a
small patch for removing the words.
Thanks, sounds fine. I think those words remain from earlier versions of
the patch committed as edd44738bc8 [1], where there used to be a fast-path
return if the ResultRelInfo was already non-NULL for the passed in index.
Makes sense -- pushed, thanks both.
Thanks for committing, Alvaro! Thanks for reviewing, Amit!
Best regards,
Etsuro Fujita
