Re: Changing baserel to foreignrel in postgres_fdw functions

Поиск
Список
Период
Сортировка
От Ashutosh Bapat
Тема Re: Changing baserel to foreignrel in postgres_fdw functions
Дата
Msg-id CAGEoWWRLuhq1w+sPxSLDek3cRG4aQZ5g2NT0LRULk3O-XuoiNA@mail.gmail.com
обсуждение исходный текст
Ответ на Re: Changing baserel to foreignrel in postgres_fdw functions  (Bruce Momjian <bruce@momjian.us>)
Список pgsql-hackers
Дерево обсуждения


On Wed, Nov 22, 2023 at 3:27 AM Bruce Momjian <bruce@momjian.us> wrote:

Should this patch be applied?

I think so. 
 
---------------------------------------------------------------------------

On Thu, Feb 15, 2018 at 06:57:50PM +0530, Ashutosh Bapat wrote:
> Hi,
> I noticed that functions is_foreign_expr(), classifyConditions() and
> appendOrderByClause() had variables/arguments named baserel when the
> relations passed to those could be join or upper relation as well.
> Here's patch renaming those as foreignrel.
 
The patch is more than 5 years old. So it might need adjustments. E.g. the appendOrderByClause() does not require the change anymore. But the other two functions still require the changes. There may be other new places that require change. I have not checked that. If we are accepting this change, I can update the patch.
 
--
Best Wishes,
Ashutosh

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

Предыдущее
От: Tomas Vondra
Дата:
Сообщение: Re: undetected deadlock in ALTER SUBSCRIPTION ... REFRESH PUBLICATION
Следующее
От: Bowen Shi
Дата:
Сообщение: [PATCH] Add CHECK_FOR_INTERRUPTS in scram_SaltedPassword loop.