Skip to content

Conversation

@erickt
Copy link
Contributor

@erickt erickt commented Sep 14, 2015

ApplicationMaster no longer has the --num-executors flag, and had an undocumented --properties-file configuration option.

cc @srowen

@vanzin
Copy link
Contributor

vanzin commented Sep 14, 2015

ok to test

@vanzin
Copy link
Contributor

vanzin commented Sep 14, 2015

LGTM although no one really should be executing this class directly.

@andrewor14
Copy link
Contributor

Yup LGTM merging into master.

@asfgit asfgit closed this in 16b6d18 Sep 14, 2015
@SparkQA
Copy link

SparkQA commented Sep 14, 2015

Test build #42440 has finished for PR 8754 at commit c7d1686.

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

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