postgres.git / summary / log / commit / refs

commit    f41042cea0619eaa812e630f87472e805b0dfdb0
Author:   Andrew Dunstan <andrew@dunslane.net>
Date:     Wed May 27 02:54:55 2015 +0000
First released in: 9.5.0, 9.4.3

    Revert "Add all structured objects passed to pushJsonbValue piecewise."
    
    This reverts commit 54547bd87f49326d67051254c363e6597d16ffda.
    
    This appears to have been a thinko on my part. I will try to come up
    wioth a better solution.


 src/backend/utils/adt/jsonb_util.c | 14 +++++---------
 1 file changed, 5 insertions(+), 9 deletions(-)

diff --git a/src/backend/utils/adt/jsonb_util.c b/src/backend/utils/adt/jsonb_util.c
index 38e1db29e5a..4d733159d06 100644
--- a/src/backend/utils/adt/jsonb_util.c
+++ b/src/backend/utils/adt/jsonb_util.c
@@ -509,8 +509,8 @@ fillJsonbValue(JsonbContainer *container, int index,
  * "raw scalar" pseudo array to append it - the actual scalar should be passed
  * next and it will be added as the only member of the array.
  *
- * All non-scalar types (jvbBinary, jbvArray and jbvObject) passed as
- * elements or values are unpacked before being added to the result.
+ * Values of type jvbBinary, which are rolled up arrays and objects,
+ * are unpacked before being added to the result.
  */
 JsonbValue *
 pushJsonbValue(JsonbParseState **pstate, JsonbIteratorToken seq,
@@ -522,18 +522,14 @@ pushJsonbValue(JsonbParseState **pstate, JsonbIteratorToken seq,
 	JsonbIteratorToken tok;
 
 	if (!jbval || (seq != WJB_ELEM && seq != WJB_VALUE) ||
-		IsAJsonbScalar(jbval))
+		jbval->type != jbvBinary)
 	{
 		/* drop through */
 		return pushJsonbValueScalar(pstate, seq, jbval);
 	}
 
-	/* unpack the data and add each piece to the pstate */
-	if (jbval->type == jbvBinary)
-		it = JsonbIteratorInit(jbval->val.binary.data);
-	else
-		it = JsonbIteratorInit(jbval);
-
+	/* unpack the binary and add each piece to the pstate */
+	it = JsonbIteratorInit(jbval->val.binary.data);
 	while ((tok = JsonbIteratorNext(&it, &v, false)) != WJB_DONE)
 		res = pushJsonbValueScalar(pstate, tok,
 								   tok < WJB_BEGIN_ARRAY ? &v : NULL);


[parent: 956cc4434c3a]