Hi, On Mon, Jan 27, 2025 at 09:30:16AM +0900, Michael Paquier wrote: > The addition of the extra GetCurrentTimestamp() in the report path > does not stress me much,
Reading at the previous messages I see how you reached this state. I also think that makes sense and that's not an issue as we are not in a hot path here. > should just do the attached, which is simpler and addresses your > use-case. Note also that the end time is acquired while the entries > are not locked in the report routines, and some tweaks in the docs and > comments. LGTM. Regards, -- Bertrand Drouvot PostgreSQL Contributors Team RDS Open Source Databases Amazon Web Services: https://aws.amazon.com
