[PATCH] pgrowlocks: Make mode names consistent with docs

Поиск
Список
Период
Сортировка
От David Cook
Тема [PATCH] pgrowlocks: Make mode names consistent with docs
Дата
Msg-id CA+dNBPNBf+FCEwohe7SH1tSks0R_G4F=tuvM=hnPs4qWiAH8vg@mail.gmail.com
обсуждение исходный текст
Ответы Re: [PATCH] pgrowlocks: Make mode names consistent with docs  (Bruce Momjian <bruce@momjian.us>)
Список pgsql-hackers
I noticed that pgrowlocks will use different names for shared locks depending on whether the locks are intermediated by a multixact or not. Particularly, if a single transaction has locked a row, it may return "For Key Share" or "For Share" in the "modes" array, while if multiple transactions have locked a row, it may return "Key Share" or "Share". The documentation of the pgrowlocks function only lists "Key Share" and "Share" as possible modes. (The four exclusive lock modes use the correct names in both cases)

The attached patch (against the master branch) fixes this discrepancy, by using "Key Share" and "Share" in the single transaction case, since that matches the documentation. I also updated the test's expected output so it passes again.

Thanks,
--David Cook
Вложения

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

Предыдущее
От: Nathan Bossart
Дата:
Сообщение: Re: ProcessStartupPacket(): database_name and user_name truncation
Следующее
От: Tom Lane
Дата:
Сообщение: Re: ProcessStartupPacket(): database_name and user_name truncation