Tomas Vondra <tomas.von...@2ndquadrant.com> writes:
>>> On 08/09/2018 07:47 PM, Alvaro Herrera wrote:
>>>> Actually, it seems to me that ApplyLogicalMappingFile is just leaking
>>>> the file descriptor for no good reason.
> I think the fix can be as simple as attached ... I'm mostly afk for the
> weekend, so I'll commit & backpatch on Monday or so.
LGTM. While you're at it, would you fix the misspelling three lines
below this?
* Check whether the TransactionOId 'xid' is in the pre-sorted array 'xip'.
^
regards, tom lane
