On 11/3/21 16:18, Tomas Vondra wrote:
> Hi,
>
> I looked at this patch today - it's pretty simple and in pretty good
> shape, I can't think of anything that'd need fixing. Perhaps the test
> might also do EXPLAIN like for other types, to verify the new index is
> actually used. But that's minor enough to handle during commit.
>
>
> I've marked this as RFC and will get it committed in a day or two.
>
Pushed, after adding some simple EXPLAIN to the regression test.
Thanks for the patch!
regards
--
Tomas Vondra
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company