bojana-db commented on code in PR #57827:
URL: https://github.com/apache/spark/pull/57827#discussion_r3896748510


##########
sql/api/src/main/scala/org/apache/spark/sql/TableValuedFunction.scala:
##########
@@ -169,6 +169,22 @@ abstract class TableValuedFunction {
    */
   def variant_explode(input: Column): Dataset[Row]
 
+  /**
+   * Separates a variant object/array into multiple rows, including nested 
fields/elements when
+   * `recursive` is true. When false, the result schema is
+   * `struct<pos int, key string, value variant>`; when true, the result 
schema is
+   * `struct<path string, pos int, key string, value variant>`.
+   *
+   * `pos` is the position within the parent, `key` is the object field name 
or NULL for arrays,
+   * and `value` is the field/element value. In recursive mode, `path` is its 
JSONPath. It ignores

Review Comment:
   It is dot notation primarily and if not representable by dot notation then 
it is written by `["..."]`, then `['...']`. Should we only keep it canonical? 
There is also an open question about the path generation. Can you share your 
opinion (https://github.com/apache/spark/pull/57827#discussion_r3808074619)? 



-- 
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]

Reply via email to