| From: | Robert Haas <robertmhaas(at)gmail(dot)com> |
|---|---|
| To: | Michael Paquier <michael(at)paquier(dot)xyz> |
| Cc: | PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: removing global variable ThisTimeLineID |
| Date: | 2021-11-10 14:52:06 |
| Message-ID: | CA+TgmoaAK3EtEt+RxRQ6KOL1UUS-iU0zWB6EtSFVqSzft-OrbQ@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Tue, Nov 9, 2021 at 7:41 PM Michael Paquier <michael(at)paquier(dot)xyz> wrote:
> On Tue, Nov 09, 2021 at 02:15:51PM -0500, Robert Haas wrote:
> > That's a good point. However, since I think newTLI is already in use
> > in some of the functions StartupXLOG() calls, and since I think it
> > would be good to use the same name in StartupXLOG() as in the
> > functions that it calls, what I'd prefer to do is rename the newTLI
> > variable in the inner scope to newReplayTLI, as in the attached v2.
>
> WFM. Thanks.
Cool. Committed that way.
--
Robert Haas
EDB: http://www.enterprisedb.com
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tom Lane | 2021-11-10 14:53:27 | Re: Removed unused import modules from tap tests |
| Previous Message | Tom Lane | 2021-11-10 14:49:35 | Re: Should AT TIME ZONE be volatile? |