Patch 0007: 1: 400 + /* 401 + * XXX: might be worthwhile to only handle oid sysattr, to reduce 402 + * overhead - it's the most common key. 403 + */
IMHO, let fix that as well. I tested this by fixing (see the attach patch) but does not found much gain on my local centos vm (of course, the pgbench load wasn't big enough). 2: How about have wrapping following condition in SearchCatCacheMiss() by unlikely(): if (IsBootstrapProcessingMode()) return NULL; 3: Can we have following assert in SearchCatCacheN() instead SearchSysCacheN(), so that we'll assert direct SearchCatCacheN() call as well? Assert(SysCache[cacheId]->cc_nkeys == <N>); Other than these concern, patch looks pretty reasonable to me. Regards, Amul
0007_ex_handle_oid.patch
Description: Binary data
-- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers