Re: [PATCH] Generic type subscription
От | Haribabu Kommi |
---|---|
Тема | Re: [PATCH] Generic type subscription |
Дата | |
Msg-id | CAJrrPGcyJxi7EUGtHvB2DCf2Zz7d6R7qem2XBpsTjpm3VMqVLg@mail.gmail.com обсуждение исходный текст |
Ответ на | Re: [PATCH] Generic type subscription (Dmitry Dolgov <9erthalion6@gmail.com>) |
Ответы |
Re: [HACKERS] [PATCH] Generic type subscription
|
Список | pgsql-hackers |
On Thu, Nov 17, 2016 at 10:56 PM, Dmitry Dolgov <9erthalion6@gmail.com> wrote:
> On 15 November 2016 at 15:03, Aleksander Alekseev <a.alekseev@postgrespro.ru> wrote:> Hello.>> I took a look on the latest -v4 patch. I would like to note that this> patch breaks a backward compatibility. For instance sr_plan extension[1]> stop to compile with errorsThank you for the feedback.Well, if we're speaking about this particular extension, if I understoodcorrectly, it fetches all parse tree nodes from Postgres and generates codeusing this information. So to avoid compilation problems I believe you need torun `make USE_PGXS=1 genparser` again (it worked for me, I don't see anymentions of `ArrayRef`).But speaking generally, I don't see how we can provide backward compatibilityfor those extensions, who are strongly coupled with implementation details ofparsing tree. I mean, in terms of interface it's mostly about to replace`ArrayRef` to `SubscriptingRef`, but I think it's better to do it in theextension code.
Moved to next CF with "needs review" status.
Regards,
Hari Babu
Fujitsu Australia
В списке pgsql-hackers по дате отправления: