Skip to content

Conversation

@gerashegalov
Copy link
Contributor

What changes were proposed in this pull request?

We propose not to hard-code the RPC port in the AM registration.

How was this patch tested?

Tested application reports from a pseudo-distributed cluster

18/04/10 14:56:21 INFO Client:
client token: N/A
diagnostics: N/A
ApplicationMaster host: localhost
ApplicationMaster RPC port: 58338
queue: default
start time: 1523397373659
final status: UNDEFINED
tracking URL: http://localhost:8088/proxy/application_1523370127531_0016/

@vanzin
Copy link
Contributor

vanzin commented Apr 11, 2018

ok to test

@SparkQA
Copy link

SparkQA commented Apr 11, 2018

Test build #89222 has finished for PR 21047 at commit cc67ccb.

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

Copy link
Contributor

@jerryshao jerryshao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@jerryshao
Copy link
Contributor

Jenkins, retest this please.

@SparkQA
Copy link

SparkQA commented Apr 16, 2018

Test build #89384 has finished for PR 21047 at commit cc67ccb.

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

@jerryshao
Copy link
Contributor

Merging to master. Thanks!

@asfgit asfgit closed this in c096493 Apr 16, 2018
@gerashegalov
Copy link
Contributor Author

thank you for review and commit @jerryshao !

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.

4 participants