Re: Clean up JumbleQuery() from query text

2023-06-26 Thread Michael Paquier
On Mon, Jun 26, 2023 at 08:51:17AM -0700, Nathan Bossart wrote: > On Mon, Jun 26, 2023 at 05:44:49PM +0900, Michael Paquier wrote: >> 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 i

Re: Clean up JumbleQuery() from query text

2023-06-26 Thread Nathan Bossart
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?

Clean up JumbleQuery() from query text

2023-06-26 Thread Michael Paquier
Hi all, 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 rathe