On Fri, Nov 16, 2018 at 03:23:55PM +0900, Michael Paquier wrote: > I was looking at this patch, and shouldn't we worry about compatibility > with plugins or utilities which look directly at what's in readRecordBuf > for the record contents? Let's not forget that the contents of > XLogReaderState are public.
And a couple of days later, committed. I did not notice first that the comments of xlogreader.h mention that a couple of areas are private, and readRecordBuf is part of that, so objection withdrawn. There is a comment in xlog.c (on top of ReadRecord) referring to readRecordBuf which has not been removed as part of 7fcbf6a4 when WAL reading has been moved to its own facility. The original comment was from 0ffe11a. So I have removed the comment at the same time. -- Michael
signature.asc
Description: PGP signature
