On 08/11/2018 04:15 PM, Tom Lane wrote: > 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'. > ^ >
Sure. -- Tomas Vondra http://www.2ndQuadrant.com PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
