Re: [HACKERS] InvokeObjectPostAlterHook() vs. CommandCounterIncrement()

2015-03-09 Thread Alvaro Herrera
Robert Haas wrote:
 On Sun, Jul 21, 2013 at 4:44 AM, Ants Aasma ants.aa...@eesti.ee wrote:
  On Jul 21, 2013 4:06 AM, Noah Misch n...@leadboat.com wrote:
  If these hooks will need to apply to a larger operation, I
  think that mandates a different means to reliably expose the before/after
  object states.
 
  I haven't checked the code to see how it would fit the API, but what about
  taking a snapshot before altering and passing this to the hook. Would there
  be other issues besides performance? If the snapshot is taken only when
  there is a hook present then the performance can be fixed later.
 
 I had the idea of finding a way to pass either the old tuple, or
 perhaps just the TID of the old tuple.  Not sure if passing a snapshot
 is better.

It seems this issue was forgotten.  Any takers?

-- 
Álvaro Herrerahttp://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training  Services


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


Re: [HACKERS] InvokeObjectPostAlterHook() vs. CommandCounterIncrement()

2013-07-22 Thread Robert Haas
On Sun, Jul 21, 2013 at 4:44 AM, Ants Aasma ants.aa...@eesti.ee wrote:
 On Jul 21, 2013 4:06 AM, Noah Misch n...@leadboat.com wrote:
 If these hooks will need to apply to a larger operation, I
 think that mandates a different means to reliably expose the before/after
 object states.

 I haven't checked the code to see how it would fit the API, but what about
 taking a snapshot before altering and passing this to the hook. Would there
 be other issues besides performance? If the snapshot is taken only when
 there is a hook present then the performance can be fixed later.

I had the idea of finding a way to pass either the old tuple, or
perhaps just the TID of the old tuple.  Not sure if passing a snapshot
is better.

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


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


Re: [HACKERS] InvokeObjectPostAlterHook() vs. CommandCounterIncrement()

2013-07-21 Thread Ants Aasma
On Jul 21, 2013 4:06 AM, Noah Misch n...@leadboat.com wrote:
 If these hooks will need to apply to a larger operation, I
 think that mandates a different means to reliably expose the before/after
 object states.

I haven't checked the code to see how it would fit the API, but what about
taking a snapshot before altering and passing this to the hook. Would there
be other issues besides performance? If the snapshot is taken only when
there is a hook present then the performance can be fixed later.

Regards,
Ants Aasma


Re: [HACKERS] InvokeObjectPostAlterHook() vs. CommandCounterIncrement()

2013-07-21 Thread Noah Misch
On Sun, Jul 21, 2013 at 11:44:51AM +0300, Ants Aasma wrote:
 On Jul 21, 2013 4:06 AM, Noah Misch n...@leadboat.com wrote:
  If these hooks will need to apply to a larger operation, I
  think that mandates a different means to reliably expose the before/after
  object states.
 
 I haven't checked the code to see how it would fit the API, but what about
 taking a snapshot before altering and passing this to the hook. Would there
 be other issues besides performance? If the snapshot is taken only when
 there is a hook present then the performance can be fixed later.

That would work.

-- 
Noah Misch
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