pg_stat_statements: remove redundant function call in pg_stat_statements_internal

Поиск
Список
Период
Сортировка
От Dong Wook Lee
Тема pg_stat_statements: remove redundant function call in pg_stat_statements_internal
Дата
Msg-id CAAcBya+szDd1Y6dJU4_dbH_Ye3=G=8O1oQGG01kv3Tpie7wELQ@mail.gmail.com
обсуждение исходный текст
Ответы Re: pg_stat_statements: remove redundant function call in pg_stat_statements_internal  (Julien Rouhaud <rjuju123@gmail.com>)
Список pgsql-hackers
Hi,

I found some redundant function calls in
pg_stat_statements.c/pg_stat_statements_internal(),
There is no need to call GetUserId() again because the value was
previously obtained.
so I propose a patch to fix it.

--- a/contrib/pg_stat_statements/pg_stat_statements.c
+++ b/contrib/pg_stat_statements/pg_stat_statements.c
@@ -1508,7 +1508,7 @@ pg_stat_statements_internal(FunctionCallInfo fcinfo,
        pgssEntry  *entry;

        /* Superusers or members of pg_read_all_stats members are allowed */
-       is_allowed_role = is_member_of_role(GetUserId(),
ROLE_PG_READ_ALL_STATS);
+       is_allowed_role = is_member_of_role(userid, ROLE_PG_READ_ALL_STATS);

        /* hash table must exist already */
        if (!pgss || !pgss_hash)

Regards,
Lee Dong Wook.

Вложения

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

Предыдущее
От: Dean Rasheed
Дата:
Сообщение: Re: Some optimisations for numeric division
Следующее
От: Julien Rouhaud
Дата:
Сообщение: Re: pg_stat_statements: remove redundant function call in pg_stat_statements_internal