| From: | Michael Paquier <michael(at)paquier(dot)xyz> |
|---|---|
| To: | Postgres hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Cc: | Joe Conway <mail(at)joeconway(dot)com> |
| Subject: | Clean up JumbleQuery() from query text |
| Date: | 2023-06-26 08:44:49 |
| Message-ID: | ZJlQAWE4COFqHuAV@paquier.xyz |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
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 rather clean up that now rather than
later.
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.
Thoughts?
--
Michael
| Attachment | Content-Type | Size |
|---|---|---|
| queryjumble-query.patch | text/x-diff | 2.6 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tatsuo Ishii | 2023-06-26 08:45:07 | Re: Row pattern recognition |
| Previous Message | Önder Kalacı | 2023-06-26 07:52:31 | Re: [Patch] Use *other* indexes on the subscriber when REPLICA IDENTITY is FULL |