Re: patch: Add missing descriptions for rmgr APIs

2022-08-29 Thread Amit Kapila
On Mon, Aug 29, 2022 at 6:19 PM kuroda.hay...@fujitsu.com wrote: > > > Your observation seems correct to me but you have not updated the > > comment for the mask. Is there a reason for the same? > > Oh, it seems that I attached wrong one. There is no reason. > PSA the newer version. > LGTM. Pushe

RE: patch: Add missing descriptions for rmgr APIs

2022-08-29 Thread kuroda.hay...@fujitsu.com
> Your observation seems correct to me but you have not updated the > comment for the mask. Is there a reason for the same? Oh, it seems that I attached wrong one. There is no reason. PSA the newer version. Best Regards, Hayato Kuroda FUJITSU LIMITED v2-0001-add-a-missing-comment.patch Descrip

Re: patch: Add missing descriptions for rmgr APIs

2022-08-29 Thread Amit Kapila
On Mon, Aug 29, 2022 at 8:48 AM kuroda.hay...@fujitsu.com wrote: > > Hi hackers, > > While reading codes related with logical decoding, > I thought that following comment in rmgrlist.h is not consistent. > > > /* symbol name, textual name, redo, desc, identify, startup, cleanup */ > > This comment

patch: Add missing descriptions for rmgr APIs

2022-08-28 Thread kuroda.hay...@fujitsu.com
Hi hackers, While reading codes related with logical decoding, I thought that following comment in rmgrlist.h is not consistent. > /* symbol name, textual name, redo, desc, identify, startup, cleanup */ This comment describes a set of APIs that the resource manager should have, but functions fo