| From: | Julien Rouhaud <rjuju123(at)gmail(dot)com> |
|---|---|
| To: | PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | cleanup & refactoring on reindexdb.c |
| Date: | 2019-05-12 09:16:28 |
| Message-ID: | CAOBaU_bSmSik_WRK9niDnm-3NkNZky6+uKxkmQwvthZvMWpS5A@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi,
As discussed in
https://www.postgresql.org/message-id/CAOBaU_Yo61RwNO3cW6WVYWwH7EYMPuexhKqufb2nFGOdunbcHw@mail.gmail.com,
current coding in reindexdb.c is error prone, and
reindex_system_catalogs() is also not really required.
I attach two patches to fix both (it could be squashed in a single
commit as both are straightforward), for upcoming v13.
| Attachment | Content-Type | Size |
|---|---|---|
| 0002-merge_reindex_system_catalogs-v1.diff | text/x-patch | 3.2 KB |
| 0001-use_enum-v1.diff | text/x-patch | 5.7 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Andrew Dunstan | 2019-05-12 13:33:16 | Re: Why is infinite_recurse test suddenly failing? |
| Previous Message | Michael Paquier | 2019-05-12 09:07:38 | Re: Wrong dead return value in jsonb_utils.c |