On Mon, 2024-04-08 at 10:24 +0200, Alvaro Herrera wrote:
> My trouble with the "copy" term is that we don't use that term
> anywhere
> in relation to WAL.
I got the term from CopyXLogRecordToWAL().
> This "copy" is in
> reality just the insertion, after it's finished. The "Result" suffix
> is intended to convey that it's the point where the operation has
> successfully finished.
That's a convincing point.
> Maybe we can add some commentary to make this clearer.
For now, I'd just suggest a comment on GetXLogInsertRecPtr() explaining
what it's returning and how that's different from logInsertResult.
In the future, it would be nice to clarify where variable names like
reservedUpto and CurrBytePos fit in. Also the external documentation
for pg_current_wal_insert_lsn() is a bit ambiguous.
> Lastly, I just noticed that I forgot credits and discussion link in
> that
> commit message. I would have attributed authorship to Bharath
> though,
> because I had not realized that this patch had actually been written
> by
> you initially[1]. My apologies.
No worries. Thank you for reviewing and committing it.
Regards,
Jeff Davis