On Fri, Jun 29, 2018 at 6:14 PM Sergei Kornilov <[email protected]> wrote:
> Hello
> Thank you for update
>
Thanks for the review.
> It may be better to use NULL as the default value at sql level.
>
Changed.
> > ereport(LOG, (errmsg("userid %u, dbid %u, queryid %ld does not exist",
> userid, dbid, queryid)));
> I think LOG level is not useful here. In common case this is server log
> only. How about WARNING? Or just ignore. Want remove row? Here is no such
> row anymore, all fine.
>
I went with removing of ereport.
> Also we can return num_remove instead of void. I think this is even
> better. But this break backward compatibility and we need something like
> pg_stat_statements_reset_1_6
>
Changed. Returning removed rows is nice than just void.
> > By default, this function can only be executed by superusers.
> Can you also update this phrase in pg_stat_statements_reset documentation?
> Beginning from 1.5 version this is not true, reset can be used by any user
> with pg_read_all_stats role.
>
Doc is updated and also separate thread is started for the doc fix [1].
Updated patch attached.
[1] -
https://www.postgresql.org/message-id/CAJrrPGcim%3DQ-7ewhuKr1n3mkeELySC-QeVHGZJJYwaaKMSJRkg%40mail.gmail.com
Regards,
Haribabu Kommi
Fujitsu Australia
0001-pg_stat_statements_reset-to-reset-specific-query-use_v2.patch
Description: Binary data
