eldenmoon commented on code in PR #39022:
URL: https://github.com/apache/doris/pull/39022#discussion_r1721160998
##########
be/src/vec/columns/column_object.cpp:
##########
@@ -1132,6 +1298,44 @@ rapidjson::Value*
find_leaf_node_by_path(rapidjson::Value& json, const PathInDat
return find_leaf_node_by_path(current, path, idx + 1);
}
+// skip empty json:
+// 1. null value as empty json
+// 2. nested array with only nulls, eg. [null. null]
Review Comment:
if we output such case, then there will be many all null elements arrays in
output json, i think it's confusing for the user.But if we take such case eg.
[null. null] as null in nested arrays, the output will be much more clear and
efficient
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
