From: | Robert Haas <rhaas(at)postgresql(dot)org> |
---|---|
To: | pgsql-committers(at)postgresql(dot)org |
Subject: | pgsql: pg_stat_statements: Widen query IDs from 32 bits to 64 bits. |
Date: | 2017-10-12 00:04:55 |
Message-ID: | E1e2Qzf-000295-3m@gemulon.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
pg_stat_statements: Widen query IDs from 32 bits to 64 bits.
This takes advantage of the infrastructure introduced by commit
81c5e46c490e2426db243eada186995da5bb0ba7 to greatly reduce the
likelihood that two different queries will end up with the same query
ID. It's still possible, of course, but whereas before it the chances
of a collision reached 25% around 50,000 queries, it will now take
more than 3 billion queries.
Backward incompatibility: Because the type exposed at the SQL level is
int8, users may now see negative query IDs in the pg_stat_statements
view (and also, query IDs more than 4 billion, which was the old
limit).
Patch by me, reviewed by Michael Paquier and Peter Geoghegan.
Discussion: http://postgr.es/m/CA+TgmobG_Kp4cBKFmsznUAaM1GWW6hhRNiZC0KjRMOOeYnz5Yw@mail.gmail.com
Branch
------
master
Details
-------
https://git.postgresql.org/pg/commitdiff/cff440d368690f94fbda1a475277e90ea2263843
Modified Files
--------------
contrib/pg_stat_statements/pg_stat_statements.c | 76 ++++++++-----------------
src/backend/executor/execParallel.c | 2 +-
src/backend/nodes/outfuncs.c | 7 ++-
src/backend/nodes/readfuncs.c | 11 +++-
src/backend/rewrite/rewriteHandler.c | 2 +-
src/include/nodes/parsenodes.h | 2 +-
src/include/nodes/plannodes.h | 2 +-
7 files changed, 42 insertions(+), 60 deletions(-)
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2017-10-12 00:13:20 | Re: pgsql: Add configure infrastructure to detect support for C99's restric |
Previous Message | Andres Freund | 2017-10-11 23:35:39 | pgsql: Use one stringbuffer for all rows printed in printtup.c. |