Re: [PATCH] Generalized JSON output functions

On 07/17/2015 10:30 AM, Alvaro Herrera wrote:
> Andrew Dunstan wrote:
>
>> 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.
> I went through all your responses over the entire thread and I couldn't
> find your argument about how this is fundamentally broken. Can you
> restate, or maybe give an archive link if I just missed it?
>
>
> (Saying "but it changes so much of the existing code" is not really a
> fundamental problem to me. I mean, it's not like the existing code is
> perfect and needs no changes.)
>

On July 13 I wrote:

> Yes, but I think the plugin is the right place to do it. What is more,
> this won't actually prevent you completely from producing
> non-ECMAScript compliant JSON, since json or jsonb values containing
> offending numerics won't be caught, AIUI. But a fairly simple to write
> function that reparsed and fixed the JSON inside the decoder would work.

The OP admitted that this was a serious flaw in his approach. In fact,
given that a json value can contain an offending numeric value, any
approach which doesn't involve reparsing is pretty much bound to fail.

cheers

andrew