Re: Bytea PL/Perl transform

From: Ivan Panchenko <wao(at)mail(dot)ru>
To: Peter Eisentraut <peter(at)eisentraut(dot)org>
Cc: pgsql-hackers(at)lists(dot)postgresql(dot)org, Greg Sabino Mullane <htamfids(at)gmail(dot)com>
Subject: Re: Bytea PL/Perl transform
Date: 2023-07-14 15:37:53
Message-ID: 1689349073.74310713@f308.i.mail.ru
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers


>Четверг, 6 июля 2023, 14:48 +03:00 от Peter Eisentraut < peter(at)eisentraut(dot)org >:

>On 22.06.23 22:56, Greg Sabino Mullane wrote:
>> * Do all of these transforms need to be their own contrib modules? So
>> much duplicated code across contrib/*_plperl already (and *plpython too
>> for that matter) ...
>The reason the first transform modules were separate extensions is that
>they interfaced between one extension (plpython, plperl) and another
>extension (ltree, hstore), so it wasn't clear where to put them without
>creating an additional dependency for one of them.
>
>If the transform deals with a built-in type, then they should just be
>added to the respective pl extension directly.
Looks reasonable. 
The new extension bytea_plperl can be easily moved into plperl now, but what should be do with the existing ones, namely jsonb_plperl and bool_plperl ?
If we leave them where they are, it would be hard to explain why some transforms are inside plperl while other ones live separately. If we move them into plperl also, wouldn’t it break some compatibility?
>

 

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2023-07-14 15:42:09 XLogSaveBufferForHint() correctness and more
Previous Message Tom Lane 2023-07-14 15:16:50 Re: In Postgres 16 BETA, should the ParseNamespaceItem have the same index as it's RangeTableEntry?