Re: Allow non-superuser to cancel superuser tasks.

Поиск
Список
Период
Сортировка
От Andres Freund
Тема Re: Allow non-superuser to cancel superuser tasks.
Дата
Msg-id zioymjtx67mvph3ppxkfisumxlhol5vtgvi6xniwfecppiyt7j@3k4qd6eraxgb
обсуждение исходный текст
Ответ на Re: Allow non-superuser to cancel superuser tasks.  (Nathan Bossart <nathandbossart@gmail.com>)
Список pgsql-hackers
Hi,

On 2024-11-22 20:44:34 -0600, Nathan Bossart wrote:
> On Fri, Nov 22, 2024 at 06:13:16PM -0500, Andres Freund wrote:
> >> -        if (procStatus && procStatus->st_backendType == B_AUTOVAC_WORKER)
> >> +        if (pgstat_get_backend_type(pid) == B_AUTOVAC_WORKER)
> > 
> > Because we already mapped the pid to a ProcNumber, it'd be cheaper to access
> > the backend status via procnumber.
> 
> D'oh, I missed that ProcNumber could be used as an index for the
> BackendStatusArray.  Is the attached more like what you are imagining?

Yes.

I'd probably add two function header comments:

1) explicit caution that this is fetching information not from the snapshot
   but "live" data
2) the return value might be out of date, that the procnumber needs to be
   valid and that the caller is responsible for permission checking

I'd also add a comment do the code saying that it's fine to bypass the
changecount mechanism, because we're reading a single 4 byte integer.

Greetings,

Andres Freund



В списке pgsql-hackers по дате отправления: