soxofaan opened a new pull request, #39516:
URL: https://github.com/apache/spark/pull/39516

   ### What changes were proposed in this pull request?
   
   See https://issues.apache.org/jira/browse/SPARK-41989 for in depth 
explanation
   
   Short summary: `pyspark/pandas/__init__.py` uses, at import time,  
`logging.warning()`  which might silently call `logging.basicConfig()`.
   So by importing `pyspark.pandas` (directly or indirectly) a user might 
unknowingly break their own logging setup (e.g. when based on  
`logging.basicConfig()` or related). `logging.getLogger(...).warning()`  does 
not trigger this behavior.
   
   
   ### Does this PR introduce _any_ user-facing change?
   
   User-defined logging setups will be more predictable.
   
   
   ### How was this patch tested?
   
   Manual testing so far. 
   I'm not sure it's worthwhile to cover this with a unit test


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