Skip to content

Conversation

@jerryshao
Copy link
Contributor

What changes were proposed in this pull request?

spark.yarn.access.namenodes configuration cannot actually reflects the usage of it, inside the code it is the Hadoop filesystems we get tokens, not NNs. So here propose to update the name of this configuration, also change the related code and doc.

How was this patch tested?

Local verification.

Change-Id: Id382d90a1b3e5cd726ee1d0badedeb433088e8b5
Change-Id: I5105b6bc7b4412ef8778d445ef5b985b1cfc0b10
@SparkQA
Copy link

SparkQA commented Jan 12, 2017

Test build #71252 has finished for PR 16560 at commit d2c651c.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@jerryshao
Copy link
Contributor Author

CC @tgravescs , please help to review, thanks a lot.

@tgravescs
Copy link
Contributor

Thanks for picking this up, will review tomorrow.

@tgravescs
Copy link
Contributor

+1

@asfgit asfgit closed this in b79cc7c Jan 17, 2017
uzadude pushed a commit to uzadude/spark that referenced this pull request Jan 27, 2017
…ate docs

## What changes were proposed in this pull request?

`spark.yarn.access.namenodes` configuration cannot actually reflects the usage of it, inside the code it is the Hadoop filesystems we get tokens, not NNs. So here propose to update the name of this configuration, also change the related code and doc.

## How was this patch tested?

Local verification.

Author: jerryshao <[email protected]>

Closes apache#16560 from jerryshao/SPARK-19179.
cmonkey pushed a commit to cmonkey/spark that referenced this pull request Feb 15, 2017
…ate docs

## What changes were proposed in this pull request?

`spark.yarn.access.namenodes` configuration cannot actually reflects the usage of it, inside the code it is the Hadoop filesystems we get tokens, not NNs. So here propose to update the name of this configuration, also change the related code and doc.

## How was this patch tested?

Local verification.

Author: jerryshao <[email protected]>

Closes apache#16560 from jerryshao/SPARK-19179.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants