Re: rmgr hooks (v2) - Mailing list pgsql-hackers

From Heikki Linnakangas
Subject Re: rmgr hooks (v2)
Date
Msg-id 49770F98.6070805@enterprisedb.com
Whole thread Raw
In response to rmgr hooks (v2)  (Simon Riggs <simon@2ndQuadrant.com>)
List pgsql-hackers
Simon Riggs wrote:
> Latest version of rmgr hooks patch for later review in current
> commitfest.

I'd like to reject this patch.

I've read through all the related threads again, and I just still don't 
see a convincing use case for it. I think that tools that let you 
introspect and modify WAL files should be written as an external 
toolkit, like pglesslog. The external indexam use case doesn't impress 
me either, and Tom seems to agree 
(http://archives.postgresql.org/message-id/24006.1221483138@sss.pgh.pa.us).

Plus there's the version incompatibility dangers. Although I think we 
could put in some safeguards and live with it, it does open new 
opportunities for confusion, so I'd rather not go there without a very 
convincing use case.

Regarding the example plugin included, for debugging purposes you could 
just compile with WAL_DEBUG, and the plugin to suppress actions for all 
but one database is clearly not ready for any real work. It only 
suppresses heapam records, replaying index updates and full-page-images 
as usual, and it requires that you know the Oid of the database,

--   Heikki Linnakangas  EnterpriseDB   http://www.enterprisedb.com


pgsql-hackers by date:

Previous
From: Andrew Chernow
Date:
Subject: Re: libpq WSACleanup is not needed
Next
From: KaiGai Kohei
Date:
Subject: Re: Column-Level Privileges