Skip to content

[SPARK-17546] [DEPLOY] start-* scripts should use hostname -f#15129

Closed
srowen wants to merge 1 commit into
apache:masterfrom
srowen:SPARK-17546
Closed

[SPARK-17546] [DEPLOY] start-* scripts should use hostname -f#15129
srowen wants to merge 1 commit into
apache:masterfrom
srowen:SPARK-17546

Conversation

@srowen

@srowen srowen commented Sep 17, 2016

Copy link
Copy Markdown
Member

What changes were proposed in this pull request?

Call hostname -f to get fully qualified host name

How was this patch tested?

Jenkins tests of course, but also verified output of command on OS X and Linux

@SparkQA

SparkQA commented Sep 17, 2016

Copy link
Copy Markdown

Test build #65532 has finished for PR 15129 at commit 5a8fb58.

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

@rxin

rxin commented Sep 18, 2016

Copy link
Copy Markdown
Contributor

LGTM.

asfgit pushed a commit that referenced this pull request Sep 18, 2016
## What changes were proposed in this pull request?

Call `hostname -f` to get fully qualified host name

## How was this patch tested?

Jenkins tests of course, but also verified output of command on OS X and Linux

Author: Sean Owen <sowen@cloudera.com>

Closes #15129 from srowen/SPARK-17546.

(cherry picked from commit 342c0e6)
Signed-off-by: Sean Owen <sowen@cloudera.com>
@srowen

srowen commented Sep 18, 2016

Copy link
Copy Markdown
Member Author

Merged to master/2.0

@asfgit asfgit closed this in 342c0e6 Sep 18, 2016
@srowen srowen deleted the SPARK-17546 branch September 18, 2016 18:17
wgtmac pushed a commit to wgtmac/spark that referenced this pull request Sep 19, 2016
## What changes were proposed in this pull request?

Call `hostname -f` to get fully qualified host name

## How was this patch tested?

Jenkins tests of course, but also verified output of command on OS X and Linux

Author: Sean Owen <sowen@cloudera.com>

Closes apache#15129 from srowen/SPARK-17546.
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