Skip to content

Conversation

@xguo27
Copy link
Contributor

@xguo27 xguo27 commented Oct 21, 2015

Minor fix on the comment

Copy link
Member

Choose a reason for hiding this comment

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

This seems fine. It occurs to me that the lines above refer to 'workers' which seems odd since this is clearly about executors, and YARN mode. Maybe those should change too.

@xguo27
Copy link
Contributor Author

xguo27 commented Oct 21, 2015

Right, let me change that too. Thx Sean!

@SparkQA
Copy link

SparkQA commented Oct 22, 2015

Test build #1939 has finished for PR 9201 at commit 5a11872.

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

@asfgit asfgit closed this in 188ea34 Oct 22, 2015
@xguo27 xguo27 deleted the SPARK-11242 branch October 22, 2015 21:10
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