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
