Re: postgres_fdw: wrong results with self join + enable_nestloop off

Поиск
Список
Период
Сортировка
От Etsuro Fujita
Тема Re: postgres_fdw: wrong results with self join + enable_nestloop off
Дата
Msg-id CAPmGK14uJJZ8s4pKjF9M-iU_YM5AoGG=p_0g-7MhF9aVdqtQDQ@mail.gmail.com
обсуждение исходный текст
Ответ на postgres_fdw: wrong results with self join + enable_nestloop off  (Nishant Sharma <nishant.sharma@enterprisedb.com>)
Ответы Re: postgres_fdw: wrong results with self join + enable_nestloop off  (Nishant Sharma <nishant.sharma@enterprisedb.com>)
Re: postgres_fdw: wrong results with self join + enable_nestloop off  (Richard Guo <guofenglinux@gmail.com>)
Список pgsql-hackers
Hi Nishant,

On Fri, Apr 14, 2023 at 8:39 PM Nishant Sharma
<nishant.sharma@enterprisedb.com> wrote:
> I debugged this issue and was able to find a fix for the same. Kindly please refer to the attached fix. With the fix
Iam able to resolve the issue. 

Thanks for the report and patch!

> What is the technical issue?
> The problem here is the use of extract_actual_clauses. Because of which the plan creation misses adding the second
conditionof AND i.e "now() < '23-Feb-2020'::timestamp" in the plan. Because it is not considered a pseudo constant and
extract_actual_clauseis passed with false as the second parameter and it gets skipped from the list. As a result that
conditionis never taken into consideration as either one-time filter (before or after) or part of SQL remote execution 
>
> Why do I think the fix is correct?
> The fix is simple, where we have created a new function similar to extract_actual_clause which just extracts all the
conditionsfrom the list with no checks and returns the list to the caller. As a result all conditions would be taken
intoconsideration in the query plan. 

I think that the root cause for this issue would be in the
create_scan_plan handling of pseudoconstant quals when creating a
foreign-join (or custom-join) plan.  Anyway, I will look at your patch
closely, first.

Best regards,
Etsuro Fujita



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

Предыдущее
От: Thomas Munro
Дата:
Сообщение: Re: OOM in hash join
Следующее
От: "Drouvot, Bertrand"
Дата:
Сообщение: Re: Synchronizing slots from primary to standby