On Fri, Feb 7, 2025 at 3:08 AM Daniel Gustafsson <dan...@yesql.se> wrote: > > > On 6 Feb 2025, at 20:00, Sergei Kornilov <s...@zsrv.org> wrote: > > > I found that the parameter type was changed by commit 19d8e23 (Ignore BRIN > > indexes when checking for HOT updates), but the comment was not updated. > > Looks like oversight. > > I agree with your analysis, unless objected to I'll apply the attached in > shortly to update the documentation.
+ * update_indexes - in successful cases this indicates the index types + * which require new index entries for this tuple AFAIK, summarising indexes may not necessarily always need a new entry. So the following sentence looks more accurate in successful cases, this indicates the types of indexes (summarising vs non-summarising) which need an update based on this tuple. The text in the parenthese would clarify "index types", but if it's too verbose, it may be omitted. -- Best Wishes, Ashutosh Bapat
