Hi!
> I'm reviewing this patch. Currently I'm trying to understand sp-gist scan deeeper, but as for now have some small
notices.
I've passed through the code one more time. Here are few more questions:
1. Logic behind division of the patch into steps is described last time 2017-01-30, but ISTM actual steps have changed
sincethan? May I ask you to write a bit about steps of the patchset?
2. The patch leaves contribs intact. Do extensions with sp-gist opclasses need to update it's behavior somehow to be
usedas-is? Or to support new functionality?
3. There is a patch about predicate locking in SP-GiST [0] Is this KNN patch theoretically compatible with predicate
locking?Seems like it is, I just want to note that this functionality may exist.
4. Scan state now have scanStack and queue. May be it's better to name scanStack and scanQueue or stack and queue?
Best regards, Andrey Borodin.
[0] https://commitfest.postgresql.org/14/1215/
