On Wed, Mar 11, 2026 at 11:10:44AM +0800, zhanghu wrote: > The doc changes look good overall. I applied the patch locally and rendered > the documentation to HTML, the new page also looks good. > > A few minor nitpicks: > > + <para> > + Name of extension for dynamically-loadable modules (e.g. > + <literal>.so</literal> on Linux). > + </para> > > I think we need a "the" before "dynamically-loadable".
Don't think this one's necessary here, actually, as it refers to a general concept around modules. > + These are useful when dealing with function and object loading > + that require specific paths to work, like paths defined in a > + <command>CREATE FUNCTION</command> or <command>LOAD</command> > + command, for example. > > The ending "for example" sounds redundant, since "like" already conveys the > meaning of "for example". WFM. -- Michael
signature.asc
Description: PGP signature
