Re: resowner module README needs update?

2021-09-15 Thread Amit Langote
On Wed, Sep 15, 2021 at 4:09 PM Michael Paquier wrote: > On Tue, Sep 14, 2021 at 07:37:28PM +0900, Michael Paquier wrote: > > No objections here. > > Done as of cae6fc2. Thank you. -- Amit Langote EDB: http://www.enterprisedb.com

Re: resowner module README needs update?

2021-09-15 Thread Michael Paquier
On Tue, Sep 14, 2021 at 07:37:28PM +0900, Michael Paquier wrote: > No objections here. Done as of cae6fc2. -- Michael signature.asc Description: PGP signature

Re: resowner module README needs update?

2021-09-14 Thread Michael Paquier
On Tue, Sep 14, 2021 at 12:18:21PM +0900, Amit Langote wrote: > Patch updated. Given the new text, I thought it might be better to > move the paragraph right next to the description of the ResourceOwner > API at the beginning of the section, because the context seems clearer > that way.

Re: resowner module README needs update?

2021-09-13 Thread Amit Langote
Thanks for looking. On Tue, Sep 14, 2021 at 9:17 AM Michael Paquier wrote: > On Mon, Sep 13, 2021 at 10:44:11PM +0900, Amit Langote wrote: > > It seems a bunch of other object/resource types have been integrated > > into the resowner mechanism since the list was last updated (2008). > > > >

Re: resowner module README needs update?

2021-09-13 Thread Michael Paquier
On Mon, Sep 13, 2021 at 10:44:11PM +0900, Amit Langote wrote: > It seems a bunch of other object/resource types have been integrated > into the resowner mechanism since the list was last updated (2008). > > Attached patch updates the list. Not sure though if we should keep > the current format

resowner module README needs update?

2021-09-13 Thread Amit Langote
Hi, I was reading src/backend/utils/resowner/README today and noticed the following paragraph that mentions the types of objects/resources *directly* supported by the module. === Currently, ResourceOwners contain direct support for recording ownership of buffer pins, lmgr locks, and catcache,