On Tue, Aug 27, 2024 at 7:22 PM Matthias van de Meent <boekewurm+postg...@gmail.com> wrote: > > Besides, I thought that you wanted me to use some new field in > > BTScanOpaque? But now you want me to use a global counter. Which is > > it? > > If you think it's important to have this info on all indexes then I'd > prefer the pgstat approach over adding a field in IndexScanDescData. > If instead you think that this is primarily important to expose for > nbtree index scans, then I'd prefer putting it in the BTSO using e.g. > the index AM analyze hook approach, as I think that's much more > elegant than this.
I agree with this analysis. I don't see why IndexScanDesc would ever be the right place for this. -- Robert Haas EDB: http://www.enterprisedb.com
