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
