On Mon, Jun 26, 2023 at 05:44:49PM +0900, Michael Paquier wrote:
> Joe has reported me offlist that JumbleQuery() includes a dependency
> to the query text, but we don't use that anymore as the query ID is
> generated from the Query structure instead.
>
> Any thoughts about the cleanup attached? But at the same time, this
> is simple and a new thing, so I'd rather clean up that now rather than
> later.
LGTM
> It is not urgent, so I am fine to postpone that after beta2 is
> released on 17~ if there are any objections to that, of course.
Even if extensions are using it, GA for v16 is still a few months away, so
IMO it's okay to apply it to v16.
--
Nathan Bossart
Amazon Web Services: https://aws.amazon.com