Dear Andrei,

I think the idea is good because the pg_stat_statements_info view cannot 
distinguish
whether the specific statement is deallocated or not.
But multiple calling of GetCurrentTimestamp() may cause some performance issues.
How about adding a configuration parameter for controlling this feature?
Or we don't not have to worry about that?


> +             if (api_version >= PGSS_V1_9)
> +             {
> +                     values[i++] = TimestampTzGetDatum(first_seen);
> +             }

I think {} is not needed here.

Best Regards,
Hayato Kuroda
FUJITSU LIMITED

Reply via email to