Etsuro Fujita <fujita.etsuro@lab.ntt.co.jp> writes:
> On 2016/07/21 16:30, Etsuro Fujita wrote:
>> One thing I'd like to discuss is GetUserMappingById/GetUserMappingId.
>> Though those functions aren't used in any places, I didn't take them
>> out, because I thought somebody else would need them someday. But
>> considering that user mapping OIDs now aren't available in RelOptInfos,
>> at least the former might be rather useless. Should we keep them in core?
> I thought a bit more about this. ISTM that there isn't much value in
> the latter either, because we can use GetUserMapping and get that OID
> from the result, instead of using the latter, so I'd vote for removing
> those functions. Attached is a patch for that.
Agreed - pushed.
regards, tom lane