From 53465075c5d7bf2a52b6af0d682219285ee219c6 Mon Sep 17 00:00:00 2001
From: jerryshao <sshao@hortonworks.com>
Date: Mon, 17 Jul 2017 13:11:30 -0700
Subject: [PATCH] [SPARK-21377][YARN] Make jars specify with --jars/--packages
load-able in AM's credential renwer
## What changes were proposed in this pull request?
In this issue we have a long running Spark application with secure HBase, which requires `HBaseCredentialProvider` to get tokens periodically, we specify HBase related jars with `--packages`, but these dependencies are not added into AM classpath, so when `HBaseCredentialProvider` tries to initialize HBase connections to get tokens, it will be failed.
Currently because jars specified with `--jars` or `--packages` are not added into AM classpath, the only way to extend AM classpath is to use "spark.driver.extraClassPath" which supposed to be used in yarn cluster mode.
So in this fix, we proposed to use/reuse a classloader for `AMCredentialRenewer` to acquire new tokens.
Also in this patch, we fixed AM cannot get tokens from HDFS issue, it is because FileSystem is gotten before kerberos logged, so using this FS to get tokens will throw exception.
## How was this patch tested?
Manual verification.
Author: jerryshao <sshao@hortonworks.com>
Closes #18616 from jerryshao/SPARK-21377.
---
.../spark/deploy/yarn/ApplicationMaster.scala | 59 ++++++++++++-------
1 file changed, 38 insertions(+), 21 deletions(-)
diff --git a/resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ApplicationMaster.scala b/resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ApplicationMaster.scala
index 4868180569..ce290c399d 100644
--- a/resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ApplicationMaster.scala
+++ b/resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ApplicationMaster.scala
@@ -90,6 +90,23 @@ private[spark] class ApplicationMaster(
@volatile private var reporterThread: Thread = _
@volatile private var allocator: YarnAllocator = _
+ private val userClassLoader = {
+ val classpath = Client.getUserClasspath(sparkConf)
+ val urls = classpath.map { entry =>
+ new URL("file:" + new File(entry.getPath()).getAbsolutePath())
+ }
+
+ if (isClusterMode) {
+ if (Client.isUserClassPathFirst(sparkConf, isDriver = true)) {
+ new ChildFirstURLClassLoader(urls, Utils.getContextOrSparkClassLoader)
+ } else {
+ new MutableURLClassLoader(urls, Utils.getContextOrSparkClassLoader)
+ }
+ } else {
+ new MutableURLClassLoader(urls, Utils.getContextOrSparkClassLoader)
+ }
+ }
+
// Lock for controlling the allocator (heartbeat) thread.
private val allocatorLock = new Object()
@@ -242,16 +259,27 @@ private[spark] class ApplicationMaster(
// If the credentials file config is present, we must periodically renew tokens. So create
// a new AMDelegationTokenRenewer
- if (sparkConf.contains(CREDENTIALS_FILE_PATH.key)) {
- // If a principal and keytab have been set, use that to create new credentials for executors
- // periodically
- val credentialManager = new YARNHadoopDelegationTokenManager(
- sparkConf,
- yarnConf,
- YarnSparkHadoopUtil.get.hadoopFSsToAccess(sparkConf, yarnConf))
-
- val credentialRenewer = new AMCredentialRenewer(sparkConf, yarnConf, credentialManager)
- credentialRenewer.scheduleLoginFromKeytab()
+ if (sparkConf.contains(CREDENTIALS_FILE_PATH)) {
+ // Start a short-lived thread for AMCredentialRenewer, the only purpose is to set the
+ // classloader so that main jar and secondary jars could be used by AMCredentialRenewer.
+ val credentialRenewerThread = new Thread {
+ setName("AMCredentialRenewerStarter")
+ setContextClassLoader(userClassLoader)
+
+ override def run(): Unit = {
+ val credentialManager = new YARNHadoopDelegationTokenManager(
+ sparkConf,
+ yarnConf,
+ YarnSparkHadoopUtil.get.hadoopFSsToAccess(sparkConf, yarnConf))
+
+ val credentialRenewer =
+ new AMCredentialRenewer(sparkConf, yarnConf, credentialManager)
+ credentialRenewer.scheduleLoginFromKeytab()
+ }
+ }
+
+ credentialRenewerThread.start()
+ credentialRenewerThread.join()
}
if (isClusterMode) {
@@ -609,17 +637,6 @@ private[spark] class ApplicationMaster(
private def startUserApplication(): Thread = {
logInfo("Starting the user application in a separate Thread")
- val classpath = Client.getUserClasspath(sparkConf)
- val urls = classpath.map { entry =>
- new URL("file:" + new File(entry.getPath()).getAbsolutePath())
- }
- val userClassLoader =
- if (Client.isUserClassPathFirst(sparkConf, isDriver = true)) {
- new ChildFirstURLClassLoader(urls, Utils.getContextOrSparkClassLoader)
- } else {
- new MutableURLClassLoader(urls, Utils.getContextOrSparkClassLoader)
- }
-
var userArgs = args.userArgs
if (args.primaryPyFile != null && args.primaryPyFile.endsWith(".py")) {
// When running pyspark, the app is run using PythonRunner. The second argument is the list
--
GitLab