On Thu, Jan 14, 2021 at 3:52 PM Fujii Masao <masao.fujii@oss.nttdata.com> wrote:
> - if (!HeapTupleIsValid(tup))
> - elog(ERROR, "cache lookup failed for user mapping %u", entry->key);
> - umform = (Form_pg_user_mapping) GETSTRUCT(tup);
> - server = GetForeignServer(umform->umserver);
> - ReleaseSysCache(tup);
> + server = GetForeignServer(entry->serverid);
>
> What about applying only the change about serverid, as a separate patch at
> first? This change itself is helpful to get rid of error "cache lookup failed"
> in pgfdw_reject_incomplete_xact_state_change(). Patch attached.
Right, we can get rid of the "cache lookup failed for user mapping"
error and also storing server oid in the cache entry is helpful for
the new functions we are going to introduce.
serverid_v1.patch looks good to me. Both make check and make
check-world passes on my system.
I will respond to other comments soon.
With Regards,
Bharath Rupireddy.
EnterpriseDB: http://www.enterprisedb.com