| From: | Andy Fan <zhihui(dot)fan1213(at)gmail(dot)com> |
|---|---|
| To: | Alexander Lakhin <exclusion(at)gmail(dot)com> |
| Cc: | Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com>, pgsql-bugs(at)lists(dot)postgresql(dot)org |
| Subject: | Re: BUG #18014: Releasing catcache entries makes schema_to_xmlschema() fail when parallel workers are used |
| Date: | 2023-08-15 23:34:02 |
| Message-ID: | CAKU4AWpqGMLLSDxUZg8RojUaBGUomFp3QmYo-b7zX02q8zxmRA@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-bugs |
Hi:
> I think that we need to determine the level where the problem that should
> be fixed is:
>
Since this thread has not been updated in the last 20 days, I'm not
sure if any people are still working on this. If not, I think considering
reducing the level of the fix is an option, to avoid the unstable testcase.
> Please look at the patch attached, that makes
> schema_get_xml_visible_tables(Oid nspid) isolated from changes in other
> namespaces.
>
I read the patch and it looks good and the change is pretty mild. so
personally +1 for this solution. Another solution suggested by me
is to force the query to go with index scan for the test case only [1],
which we get the similar result but it just works in this test case.
I think Alexander's method is better, I just put my findings here just
in case I missed something.
--
Best Regards
Andy Fan
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Michael Paquier | 2023-08-16 00:00:43 | Re: BUG #18057: unaccent removes intentional spaces |
| Previous Message | PG Bug reporting form | 2023-08-15 19:54:57 | BUG #18057: unaccent removes intentional spaces |