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

    https://github.com/apache/spark/pull/9313#discussion_r43347726
  
    --- Diff: python/pyspark/context.py ---
    @@ -806,6 +806,24 @@ def addPyFile(self, path):
                 import importlib
                 importlib.invalidate_caches()
     
    +    def addJar(self, path):
    +        """
    +        Adds a JAR dependency for all tasks to be executed on this 
SparkContext in the future.
    +        The `path` passed can be either a local file, a file in HDFS (or 
other Hadoop-supported
    +        filesystems), an HTTP, HTTPS or FTP URI, or local:/path for a file 
on every worker node.
    +        """
    +        self._jsc.sc().addJar(path)
    +        self._jvm.PythonUtils.updatePrimaryClassLoader(self._jsc)
    +
    +    def _loadClass(self, className):
    +        """
    +        .. note:: Experimental
    +
    +        Loads a JVM class using the MutableClass loader used by spark.
    +        This function exists because Py4J uses a different class loader.
    +        """
    +        
self._jvm.java.lang.Thread.currentThread().getContextClassLoader().loadClass(className)
    --- End diff --
    
    In Utils.getContextOrSparkClassLoader, it seems like the context 
classloader can sometimes be null.  Is that possible here as well?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to