Hi, Tom Lane <t...@sss.pgh.pa.us> writes: > For the record, I think this patch is a waste of manpower and we should > rely on dtrace/systemtap. However, if we are going to make our own > homegrown substitute for those facilities, a minimum requirement should > be that it uses the dtrace macros already put into the sources, rather > than expecting that it gets to clutter the code some more with its own > set of tracing markers.
I think going from this form of the patch to reusing dtrace macros means providing another entirely different patch to solve the issue (even if the system view and its support function could remain about the same), so I've updated the patch commit fest status to rejected. Regards, -- dim -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
