Re: [PATCH] Add support function for containment operators

Поиск
Список
Период
Сортировка
От Kim Johan Andersson
Тема Re: [PATCH] Add support function for containment operators
Дата
Msg-id 4cdc292e-3234-a03d-107d-31cf8b721cd4@kimmet.dk
обсуждение исходный текст
Ответ на Re: [PATCH] Add support function for containment operators  (Laurenz Albe <laurenz.albe@cybertec.at>)
Ответы Re: [PATCH] Add support function for containment operators  (Laurenz Albe <laurenz.albe@cybertec.at>)
Список pgsql-hackers
On 07-07-2023 13:20, Laurenz Albe wrote:
> I wrote:
>> You implement both "SupportRequestIndexCondition" and "SupportRequestSimplify",
>> but when I experimented, the former was never called.  That does not
>> surprise me, since any expression of the shape "expr <@ range constant"
>> can be simplified.  Is the "SupportRequestIndexCondition" branch dead code?
>> If not, do you have an example that triggers it?

I would think it is dead code, I came to the same conclusion. So we can 
drop SupportRequestIndexCondition, since the simplification happens to 
take care of everything.


> I had an idea about this:
> So far, you only consider constant ranges.  But if we have a STABLE range
> expression, you could use an index scan for "expr <@ range", for example
> Index Cond (expr >= lower(range) AND expr < upper(range)).
> 

I will try to look into this. Originally that was what I was hoping for, 
but didn't see way of going about it.

Thanks for your comments, I will also look at the locale-related 
breakage you spotted.

    Regards,
        Kimjand



В списке pgsql-hackers по дате отправления:

Предыдущее
От: Amit Kapila
Дата:
Сообщение: Re: [Patch] Use *other* indexes on the subscriber when REPLICA IDENTITY is FULL
Следующее
От: Alena Rybakina
Дата:
Сообщение: Re: Problems with estimating OR conditions, IS NULL on LEFT JOINs