Re: remaining sql/json patches
| От | jian he |
|---|---|
| Тема | Re: remaining sql/json patches |
| Дата | |
| Msg-id | CACJufxGzkVSmAT-estwsCgKE-UvMWeGWRZa5J0ktr_3VrMdKrg@mail.gmail.com обсуждение исходный текст |
| Ответ на | Re: remaining sql/json patches (jian he <jian.universality@gmail.com>) |
| Ответы |
Re: remaining sql/json patches
|
| Список | pgsql-hackers |
one more issue.
+ case JSON_VALUE_OP:
+ /* Always omit quotes from scalar strings. */
+ jsexpr->omit_quotes = (func->quotes == JS_QUOTES_OMIT);
+
+ /* JSON_VALUE returns text by default. */
+ if (!OidIsValid(jsexpr->returning->typid))
+ {
+ jsexpr->returning->typid = TEXTOID;
+ jsexpr->returning->typmod = -1;
+ }
by default, makeNode(JsonExpr), node initialization,
jsexpr->omit_quotes will initialize to false,
Even though there was no implication to the JSON_TABLE patch (probably
because coerceJsonFuncExprOutput), all tests still passed.
based on the above comment, and the regress test, you still need do (i think)
`
jsexpr->omit_quotes = true;
`
В списке pgsql-hackers по дате отправления: