diff options
author | Andrew Or <andrew@databricks.com> | 2014-12-04 21:53:38 -0800 |
---|---|---|
committer | Andrew Or <andrew@databricks.com> | 2014-12-04 21:53:38 -0800 |
commit | 87437df036305c6f467dc6e0bf7891d27d266a6b (patch) | |
tree | eb9fdcd735a3df64208ffe5450e7777731609f26 | |
parent | ca379039f701e423fa07933db4e063cb85d0236a (diff) | |
download | spark-87437df036305c6f467dc6e0bf7891d27d266a6b.tar.gz spark-87437df036305c6f467dc6e0bf7891d27d266a6b.tar.bz2 spark-87437df036305c6f467dc6e0bf7891d27d266a6b.zip |
Revert "[HOT FIX] [YARN] Check whether `/lib` exists before listing its files"
This reverts commit 90ec643e9af4c8bbb9000edca08c07afb17939c7.
-rw-r--r-- | yarn/common/src/main/scala/org/apache/spark/deploy/yarn/ClientBase.scala | 27 |
1 files changed, 12 insertions, 15 deletions
diff --git a/yarn/common/src/main/scala/org/apache/spark/deploy/yarn/ClientBase.scala b/yarn/common/src/main/scala/org/apache/spark/deploy/yarn/ClientBase.scala index 290d9943a5..8e4360ea44 100644 --- a/yarn/common/src/main/scala/org/apache/spark/deploy/yarn/ClientBase.scala +++ b/yarn/common/src/main/scala/org/apache/spark/deploy/yarn/ClientBase.scala @@ -243,21 +243,18 @@ private[spark] trait ClientBase extends Logging { val libsURI = new URI(libsDir) val jarLinks = ListBuffer.empty[String] if (libsURI.getScheme != LOCAL_SCHEME) { - val localPath = getQualifiedLocalPath(libsURI) - val localFs = FileSystem.get(localPath.toUri, hadoopConf) - if (localFs.exists(localPath)) { - val jars = localFs.listFiles(localPath, /* recursive */ false) - while (jars.hasNext) { - val jar = jars.next() - val name = jar.getPath.getName - if (name.startsWith("datanucleus-")) { - // copy to remote and add to classpath - val src = jar.getPath - val destPath = copyFileToRemote(dst, src, replication) - distCacheMgr.addResource(localFs, hadoopConf, destPath, - localResources, LocalResourceType.FILE, name, statCache) - jarLinks += name - } + val localURI = getQualifiedLocalPath(libsURI).toUri() + val jars = FileSystem.get(localURI, hadoopConf).listFiles(new Path(localURI.getPath), false) + while (jars.hasNext) { + val jar = jars.next() + val name = jar.getPath.getName + if (name.startsWith("datanucleus-")) { + // copy to remote and add to classpath + val src = jar.getPath + val destPath = copyFileToRemote(dst, src, replication) + distCacheMgr.addResource(fs, hadoopConf, destPath, + localResources, LocalResourceType.FILE, name, statCache) + jarLinks += name } } } else { |