Hi Sawada-san and Chao-san, Thank you both for your continued reviews and feedback on this patch.I agree with both points that could be misleading. I've updated the message as follows:
memory usage: dead item storage %.2f MB accumulated across %d resets (limit %.2f MB each) This wording: - Use both "memory usage:" prefix and "dead item storage" to be consistent with other existing log lines (buffer usage:, WAL usage:, system usage:) and to clarify what kind of memory we're reporting - Clearly indicate that the value is a cumulative sum of snapshots, not actual memory allocation - Use "across %d resets" to show the relationship between the accumulated value and reset count - Keep "limit %.2f MB each" to indicate the per-reset memory limit Please find attached the v11 patch. Regards, Tatsuya Kawata
v11-0001-Add-dead-items-memory-usage-to-VACUUM-VERBOSE-an.patch
Description: Binary data
