On 07/17/2015 10:11 AM, Andrew Dunstan wrote:
>
> On 07/17/2015 08:20 AM, Shulgin, Oleksandr wrote:
>
>
>> > This patch makes Postgres core more complex
>>
>> Yes, it does. But, that was not the purpose, obviously. :-)
>>
>> > while not really solving the problem in Javascript.
>>
>> It still allows for less risk of silent data corruption on the js side.
>>
>>
>
> I have already pointed out how this patch is fundamentally broken. You
> can achieve your aims by a fairly small amount of code inside your
> logical decoder, and with no core code changes whatsoever. So I'm
> puzzled why we are even still debating this broken design.
Incidentally, this doesn't look acceptable anyway:
> ! es->json_cxt.value(&es->json_cxt, num, JSONTYPE_NUMERIC,
> ! NUMERICOID, 1702 /* numeric_out */);
We don't hardcode function oids elsewhere. So this is also something
that makes the patch unacceptable.
cheers
andrew