From: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> |
---|---|
To: | Li Japin <japinli(at)hotmail(dot)com> |
Cc: | pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Remove cache_plan argument comments to ri_PlanCheck |
Date: | 2020-11-25 05:45:38 |
Message-ID: | CAA4eK1KeMi=wLgF+HcP1FS5p-1F+hQxy=V2UGLCsF5MmkgBBLQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Tue, Nov 24, 2020 at 7:26 PM Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> wrote:
>
> On Tue, Nov 24, 2020 at 4:46 PM Li Japin <japinli(at)hotmail(dot)com> wrote:
> >
> > Hi, hackers
> >
> > I found that the cache_plan argument to ri_PlanCheck already been remove since
> > 5b7ba75f7ff854003231e8099e3038c7e2eba875. I think we can remove the comments
> > tor cache_plan to ri_PlanCheck.
> >
> > diff --git a/src/backend/utils/adt/ri_triggers.c b/src/backend/utils/adt/ri_triggers.c
> > index 7e2b2e3dd6..02b1a3868f 100644
> > --- a/src/backend/utils/adt/ri_triggers.c
> > +++ b/src/backend/utils/adt/ri_triggers.c
> > @@ -2130,9 +2130,6 @@ InvalidateConstraintCacheCallBack(Datum arg, int cacheid, uint32 hashvalue)
> >
> > /*
> > * Prepare execution plan for a query to enforce an RI restriction
> > - *
> > - * If cache_plan is true, the plan is saved into our plan hashtable
> > - * so that we don't need to plan it again.
> > */
> > static SPIPlanPtr
> > ri_PlanCheck(const char *querystr, int nargs, Oid *argtypes,
> >
>
> Your patch looks good to me.
>
Pushed!
--
With Regards,
Amit Kapila.
From | Date | Subject | |
---|---|---|---|
Next Message | Craig Ringer | 2020-11-25 06:16:53 | Re: [PATCH] LWLock self-deadlock detection |
Previous Message | Fujii Masao | 2020-11-25 05:32:05 | Re: A few new options for CHECKPOINT |