ueshin commented on code in PR #41948: URL: https://github.com/apache/spark/pull/41948#discussion_r1267273028
########## python/pyspark/sql/connect/udtf.py: ########## @@ -33,8 +33,8 @@ ) from pyspark.sql.connect.types import UnparsedDataType from pyspark.sql.connect.utils import get_python_ver -from pyspark.sql.udtf import UDTFRegistration as PySparkUDTFRegistration -from pyspark.sql.udtf import _validate_udtf_handler +from pyspark.sql.udtf import UDTFRegistration as PySparkUDTFRegistration, _validate_udtf_handler +from pyspark.sql.udtf import AnalyzeArgument, AnalyzeResult # noqa: F401 Review Comment: I think it's still good to have this here for a case when a user wants to directly use `connect` package. ```py from pyspark.sql.connect.udtf import AnalyzeArgument, AnalyzeResult ``` -- 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]
