Re: Index AM API cleanup

On 27.11.24 13:57, Peter Eisentraut wrote:
> I think, however, that we should rename RowCompareType.  Otherwise, it's
> just going to be confusing forevermore.  I suggest to rename it simply
> to CompareType.

> I'm going to try to code up the gist support on top of this patch set to
> make sure that it will fit well.  I'll report back.

Here is a patch set in that direction. It renames RowCompareType to
CompareType and updates the surrounding commentary a bit. And then I'm
changing the gist strategy mapping to use the CompareType values instead
of the RT* strategy numbers. Seeing this now, I like this a lot better
than what we have now, because it makes it clearer in the API and the
code what is a real strategy number and what's a different kind of
thing. (This isn't entirely the above-mentioned integration of the gist
support into your patch set yet, but it's a meaningful part of it.)