Horton

Re: queryId constant squashing does not support prepared statements

Michael Paquier <michael@paquier.xyz>

View thread
From: Michael Paquier <michael@paquier.xyz>
To: Álvaro Herrera <alvherre@kurilemu.de>
Cc: Sami Imseih <samimseih@gmail.com>, Dmitry Dolgov <9erthalion6@gmail.com>, Junwang Zhao <zhjwpku@gmail.com>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2025-06-11T00:32:27Z
Lists: pgsql-hackers

Commits

Same data as JSON: GET /api/v1/messages/:b64id/commits the thread's linked commits as JSON, with link sources. API reference →

  1. Fix typo in comment

  2. Make query jumbling also squash PARAM_EXTERN params

  3. Fix squashing algorithm for query texts

  4. pg_stat_statements: Fix parameter number gaps in normalized queries

On Tue, Jun 10, 2025 at 07:25:27PM +0200, Alvaro Herrera wrote:
> On 2025-Jun-10, Michael Paquier wrote:
>> I think that this is can be reproduced by
>> -DWRITE_READ_PARSE_PLAN_TREES -DCOPY_PARSE_PLAN_TREES
>> -DRAW_EXPRESSION_COVERAGE_TEST that I always include in my builds.
>> The freebsd task uses the same with debug_copy_parse_plan_trees=on,
>> debug_write_read_parse_plan_trees=on and
>> debug_raw_expression_coverage_test=on.
> 
> Ah, of course, I forgot to add read/write support for A_Expr.  Pushed a
> new one, running at
> https://cirrus-ci.com/build/6249249819590656

Ah, right.  I completely forgot that we have a custom read/write
function for this node.  Yes, things should be OK once the function is
updated.  Thanks.
--
Michael