From: | Michael Paquier <michael(at)paquier(dot)xyz> |
---|---|
To: | Postgres hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Cc: | Andres Freund <andres(at)anarazel(dot)de> |
Subject: | Replace some %llu remnants in the tree |
Date: | 2025-06-09 03:59:20 |
Message-ID: | aEZcGCnYFq642q8k@paquier.xyz |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hi all,
While hacking a different patch, I've noticed that a couple of %llu
did not get the PRIu64 call in the AIO code, and I don't see why we
could not switch them. These have been introduced in commits that got
into the tree after Peter's 15a79c73111f.
A couple remain even after the attached, which have been left off by
Peter for the same reasons as the ones I am guessing here:
- launch_backend.c for paramHandle, does not seem worth it.
- ecpglib/execute.c, when storing some input, which does not seem
worth bothering either.
- reconstruct.c, offset handling.
- pg_backup_tar.c, ftello() result and some consistency with the
surroundings.
- pg_verifybackup.c and astreamer_verify.c, for Sizes.
That's not necessarily mandatory for v18, for sure, but as this is new
code we could as well clean it up before forking the next stable
branch.
Comments or opinions?
--
Michael
Attachment | Content-Type | Size |
---|---|---|
0001-Replace-llu-by-PRIu64-in-AIO-io_uring-code.patch | text/x-diff | 1.0 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | jian he | 2025-06-09 03:59:34 | Re: doc pg_constraint.convalidated column description need update |
Previous Message | shveta malik | 2025-06-09 03:57:32 | Re: POC: enable logical decoding when wal_level = 'replica' without a server restart |