On Mon, Apr 15, 2019 at 11:06:18AM -0400, Tom Lane wrote:
> Hmm. The second, duplicate assignment is surely pointless, but I think
> that setting the ctx as the private_data is a good idea. It hardly seems
> out of the question that it might be needed in future.
Agreed that we should keep the assignment done with
XLogReaderAllocate(). I have committed the patch which removes the
useless assignment though.
--
Michael