Re: Statistics Import and Export

Поиск
Список
Период
Сортировка
От jian he
Тема Re: Statistics Import and Export
Дата
Msg-id CACJufxFVQ-qWU4aU0KJDJeN1_5XEh3nhey5Vw2V5zazts7PL=A@mail.gmail.com
обсуждение исходный текст
Ответ на Re: Statistics Import and Export  (Jeff Davis <pgsql@j-davis.com>)
Список pgsql-hackers
hi.
now stats_lock_check_privileges comments need to change?

 * Lock relation in ShareUpdateExclusive mode, check privileges, and close the
 * relation (but retain the lock).

since the above comments will not be true for RELKIND_INDEX.
Yes, there are comments within the function explaining the exception.


maybe we can incorporate the following
    /*
     * For indexes, we follow what do_analyze_rel() does so as to avoid any
     * deadlocks with analyze/vacuum, which is to take out a
     * ShareUpdateExclusive on table/matview first and only after that take
     * a AccessShareLock on the index itself.
     */
comments into stats_lock_check_privileges comments section.

other than that, the patch fix looks good to me.



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