| From: | Michael Paquier <michael(at)paquier(dot)xyz> |
|---|---|
| To: | Haribabu Kommi <kommi(dot)haribabu(at)gmail(dot)com> |
| Cc: | pgsql-hackers(at)lists(dot)postgresql(dot)org, Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>, Dave Page <dpage(at)pgadmin(dot)org>, Simon Riggs <simon(at)2ndquadrant(dot)com>, Fujii Masao <masao(dot)fujii(at)gmail(dot)com> |
| Subject: | Re: Revoke execution permission of pg_stat_statements_reset() from pg_read_all_stats role |
| Date: | 2018-09-24 15:44:19 |
| Message-ID: | 20180924154419.GD1103@paquier.xyz |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Mon, Sep 24, 2018 at 11:08:14AM +1000, Haribabu Kommi wrote:
> In commit 25fff40798 the execute permission of pg_stat_statements_reset()
> is provided to pg_read_all_stats role in [1].
>
> The execute permissions grant to pg_read_all_stats concern is raised in [2]
> during the discussion of supporting different methods of reset the stats,
> instead of resetting all.
>
> Here I attached the patch that reverts the permission grant as per the
> discussion
> in [3].
This should be back-patched. Any opinions about bumping up this
extension version in back-branches like what has been done in 53b79ab4?
--
Michael
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Alvaro Herrera | 2018-09-24 15:57:12 | Re: Multiple primary key on partition table? |
| Previous Message | Michael Paquier | 2018-09-24 15:39:47 | Re: New function pg_stat_statements_reset_query() to reset statistics of a specific query |