Github user jerryshao commented on a diff in the pull request:

    https://github.com/apache/spark/pull/20404#discussion_r164364573
  
    --- Diff: python/pyspark/sql/session.py ---
    @@ -225,6 +230,9 @@ def __init__(self, sparkContext, jsparkSession=None):
             if SparkSession._instantiatedSession is None \
                     or SparkSession._instantiatedSession._sc._jsc is None:
                 SparkSession._instantiatedSession = self
    +            if self._jvm.SparkSession.getDefaultSession().isEmpty() \
    +                    or not 
jsparkSession.equals(self._jvm.SparkSession.getDefaultSession().get()):
    +                
self._jvm.SparkSession.setDefaultSession(self._jsparkSession)
    --- End diff --
    
    @felixcheung has concern about simply overwriting the default session.


---

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

Reply via email to