Heikki Linnakangas <hlinnaka(at)iki(dot)fi> writes: > In init_sql_fcache(), one comment says that the junkfilter is > responsible for injecting NULLs for dropped columns, and a later comment > says that the junk filter gets "rid of any dropped columns". That seems > contradictory; which is it? Or does "get rid of" mean "set to NULL"?
Yeah, the second comment is sloppily worded; the first one is more accurate.
> Other than that, looks good to me.
Thanks for reviewing! I'll fix that comment and push.