zhengruifeng commented on code in PR #41505:
URL: https://github.com/apache/spark/pull/41505#discussion_r1222575032


##########
python/pyspark/sql/functions.py:
##########
@@ -9929,6 +10078,26 @@ def map_zip_with(
     return _invoke_higher_order_function("MapZipWith", [col1, col2], [f])
 
 
+def str_to_map(
+    col1: "ColumnOrName",
+    col2: Union["ColumnOrName", str],
+    col3: Union["ColumnOrName", str],
+) -> Column:

Review Comment:
   sounds good, so let us use `ColumnOrName`, which treats `str` arguments as 
column names. @panbingkun 
   
   another advantage is that it can support non-foldable expressions in the 
future



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