Hi Sergei!
Thank you for your review.
On Tue, 2023-03-21 at 23:18 +0300, Sergei Kornilov wrote:
> -- Don't want this to be available to non-superusers.
> REVOKE ALL ON FUNCTION pg_stat_statements_reset(Oid, Oid, bigint,
> boolean) FROM PUBLIC;
>
> should be added to the upgrade script
Indeed.
> Also, shouldn't we first do:
>
> /* First we have to remove them from the extension */
> ALTER EXTENSION pg_stat_statements DROP VIEW ..
> ALTER EXTENSION pg_stat_statements DROP FUNCTION ..
>
> like in previous upgrade scripts?
It was discussed few messages earlier in this thread. We've decided
that those are unnecessary in upgrade script.
> > + Time at which min/max statistics gathering started for this
> > + statement
>
> I think it would be better to explicitly mention in the documentation
> all 4 fields for which minmax_stats_since displays the time.
Agreed.
New version is attached.
regards, Andrei