Re: bug in plancache.c

yamt@mwd.biglobe.ne.jp (YAMAMOTO Takashi) writes:
> GetCachedPlan can pass the 'qlist' to the planner twice.
> if i understand the code correctly, it's unsafe because the planner is
> destructive wrt the input tree.  for my application, it often causes
> a crash in executor.

Good catch, thanks!  (But a fix like this really requires a comment IMO.)

            regards, tom lane