On 2021/01/14 20:36, Bharath Rupireddy wrote:
On Thu, Jan 14, 2021 at 3:52 PM Fujii Masao <masao.fu...@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.
Thanks for the check! I pushed the patch.
I will respond to other comments soon.
Thanks!
Regards,
--
Fujii Masao
Advanced Computing Technology Center
Research and Development Headquarters
NTT DATA CORPORATION
