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.1221483...@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

--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to