You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by "HyukjinKwon (via GitHub)" <gi...@apache.org> on 2023/02/20 04:26:38 UTC

[GitHub] [spark] HyukjinKwon commented on a diff in pull request #40067: [SPARK-42476][CONNECT][DOCS] Complete Spark Connect API reference

HyukjinKwon commented on code in PR #40067:
URL: https://github.com/apache/spark/pull/40067#discussion_r1111443135


##########
python/pyspark/sql/connect/client.py:
##########
@@ -203,6 +203,8 @@ def metadata(self) -> Iterable[Tuple[str, str]]:
         parameters will be converted to metadata except ones that are explicitly used
         by the channel.
 
+        .. versionadded:: 3.4.0

Review Comment:
   `versionadded:: 3.4.0` is in the class already so it's fine



##########
python/pyspark/sql/connect/client.py:
##########
@@ -440,8 +450,12 @@ def register_udf(
         eval_type: int = PythonEvalType.SQL_BATCHED_UDF,
         deterministic: bool = True,
     ) -> str:
-        """Create a temporary UDF in the session catalog on the other side. We generate a
-        temporary name for it."""
+        """
+        Create a temporary UDF in the session catalog on the other side. We generate a
+        temporary name for it.
+
+        .. versionadded:: 3.4.0

Review Comment:
   ditto let's remove them



-- 
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: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org