Kohei KaiGai <kai...@kaigai.gr.jp> writes:
> The origin of matter is, as you mentioned, collation to be used for system
> catalog scan when we reference it via syscache.
> So, the following chunk should be added, as I did in the userspace access
> vector patch - part.1.
> @@ -934,8 +935,7 @@ CatalogCacheInitializeCache(CatCache *cache)
> /* Fill in sk_strategy as well --- always standard equality */
> cache->cc_skey[i].sk_strategy = BTEqualStrategyNumber;
> cache->cc_skey[i].sk_subtype = InvalidOid;
> - /* Currently, there are no catcaches on collation-aware data types
> */
> - cache->cc_skey[i].sk_collation = InvalidOid;
> + cache->cc_skey[i].sk_collation = DEFAULT_COLLATION_OID;
I removed such a hunk from a previous patch of yours, and I don't like
it any better this time. This is just a hack that will result in
masking bugs.
Consider using a non-collation-aware datatype instead, such as NAME.
regards, tom lane
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
