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.


Reply via email to