bjornjorgensen commented on a change in pull request #34389:
URL: https://github.com/apache/spark/pull/34389#discussion_r737336464



##########
File path: python/pyspark/pandas/groupby.py
##########
@@ -1199,6 +1200,10 @@ def pandas_apply(pdf: pd.DataFrame, *a: Any, **k: Any) 
-> Any:
 
         if should_infer_schema:
             # Here we execute with the first 1000 to get the return type.
+            log_advice(
+                "If the type hints is not specified for `grouby.apply`, "
+                "it could be expensive for inferring the type internally."

Review comment:
       Can we change "it could be expensive for inferring the type internally." 
   to "it is expensive to infer the data type internally." 
   




-- 
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-unsubscr...@spark.apache.org

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



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

Reply via email to