Craig Ringer <ring...@ringerc.id.au> writes: > OK, opr_sanity was failing because I added the value_to_json(text) alias > to ensure that:
> value_to_json('some_literal') > worked, following the same approach as quote_literal(anyelement) and > quote_literal(text). That should be reasonable, right? No, it isn't. What you're proposing is to let opr_sanity think that text and anyelement are interchangeable to C functions, which is so far from reality as to be ludicrous. That would be seriously damaging to its ability to detect errors. But more to the point, your analogy to quote_literal is faulty anyway. If you looked at that, what you'd find is that only quote_literal(text) is a C function. The other one is a SQL wrapper around a coercion to text followed by the C function. I rather imagine that the definition as you have it would crash on, say, value_to_json(42). regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers