You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2019/01/30 14:17:22 UTC

[GitHub] gaborgsomogyi opened a new pull request #23698: [SPARK-26766][CORE] Remove the list of filesystems from HadoopDelegationTokenProvider.obtainDelegationTokens

gaborgsomogyi opened a new pull request #23698: [SPARK-26766][CORE] Remove the list of filesystems from HadoopDelegationTokenProvider.obtainDelegationTokens
URL: https://github.com/apache/spark/pull/23698
 
 
   ## What changes were proposed in this pull request?
   
   Delegation token providers interface now has a parameter `fileSystems` but this is needed only for `HadoopFSDelegationTokenProvider`.
   
   In this PR I've addressed this issue in the following way:
   * Removed `fileSystems` parameter from `HadoopDelegationTokenProvider`
   * Moved `YarnSparkHadoopUtil.hadoopFSsToAccess` into `HadoopFSDelegationTokenProvider`
   * Moved `spark.yarn.stagingDir` into core
   * Moved `spark.yern.access.namenodes` into core and renamed to `spark.kerberos.access.namenodes`
   * Moved `spark.yarn.access.hadoopFileSystems` into core and renamed to `spark.kerberos.access.hadoopFileSystems`
   
   ## How was this patch tested?
   
   Existing unit tests.
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

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