Re: [HACKERS] Early Setup of instrumentation information in pg_stat_statements

2015-02-06 Thread Robert Haas
On Thu, Feb 5, 2015 at 10:28 AM, Amit Kapila wrote: > Currently in pg_stat_statements, the setup to track > instrumentation/totaltime information is done after > ExecutorStart(). Can we do this before ExecutorStart()? > In particular, I am referring to below code: > > static void > pgss_ExecutorS

Re: [HACKERS] Early Setup of instrumentation information in pg_stat_statements

2015-02-05 Thread Amit Kapila
On Thu, Feb 5, 2015 at 8:58 PM, Amit Kapila wrote: > > Currently in pg_stat_statements, the setup to track > instrumentation/totaltime information is done after > ExecutorStart(). Can we do this before ExecutorStart()? On further analyzing, I found that currently it is done after ExecutorStart()

[HACKERS] Early Setup of instrumentation information in pg_stat_statements

2015-02-05 Thread Amit Kapila
Currently in pg_stat_statements, the setup to track instrumentation/totaltime information is done after ExecutorStart(). Can we do this before ExecutorStart()? In particular, I am referring to below code: static void pgss_ExecutorStart(QueryDesc *queryDesc, int eflags) { .. standard_ExecutorStart