Please find attached the patches wanted: - 20130317_reindexdb_concurrently.patch, adding an option -c/--concurrently to reindexdb Note that I added an error inside reindexdb for options "-s -c" as REINDEX CONCURRENTLY does not support SYSTEM. - 20130317_dump_only_valid_index.patch, a 1-line patch that makes pg_dump not take a dump of invalid indexes. This patch can be backpatched to 9.0.
On Sun, Mar 17, 2013 at 3:31 AM, Michael Paquier <[email protected]>wrote: > On 2013/03/17, at 0:35, Fujii Masao <[email protected]> wrote: > > > On Wed, Mar 13, 2013 at 9:04 PM, Michael Paquier > > I found pg_dump dumps even the invalid index. But pg_dump should > > ignore the invalid index? > > This problem exists even without REINDEX CONCURRENTLY patch. So we might > need to > > implement the bugfix patch separately rather than including the bugfix > > code in your patches. > > Probably the backport would be required. Thought? > Hum... Indeed, they shouldn't be included... Perhaps this is already known? > Note that there have been some recent discussions about that. This *problem* also concerned pg_upgrade. http://www.postgresql.org/message-id/[email protected] -- Michael
20130317_reindexdb_concurrently.patch
Description: Binary data
20130317_dump_only_valid_index.patch
Description: Binary data
-- Sent via pgsql-hackers mailing list ([email protected]) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
