Skip to content

Conversation

@Sephiroth-Lin
Copy link
Contributor

No description provided.

@srowen
Copy link
Member

srowen commented Apr 13, 2015

I don't object to this, although most of the YARN-related content here is almost all redundant, since it doesn't change between Python and Scala/Java, and this is a YARN-related doc. The docs are kind of out of order now since the explanation of SparkPi followed the python example. I am not sure this adds enough value.

@srowen
Copy link
Member

srowen commented Apr 13, 2015

(That is, I don't think this is quite what https://issues.apache.org/jira/browse/SPARK-5689 is getting at; a simple statement of what can be run now might be sufficient rather than duplicating this much.)

@SparkQA
Copy link

SparkQA commented Apr 13, 2015

Test build #30160 has finished for PR 5490 at commit 97ba6a8.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.
  • This patch does not change any dependencies.

@SparkQA
Copy link

SparkQA commented Apr 13, 2015

Test build #30161 has finished for PR 5490 at commit 8ca1efe.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.
  • This patch does not change any dependencies.

@SparkQA
Copy link

SparkQA commented Apr 13, 2015

Test build #30163 has finished for PR 5490 at commit 908bb72.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.
  • This patch does not change any dependencies.

@tgravescs
Copy link
Contributor

For this I was actually thinking just making a statement about specifically what can be run in either mode. Perhaps around paragraph on line 215 where we discuss the two modes.

interactive shell(s) (both pyspark, spark-sql, spark-shell) can only be in used client mode. That might be it since we recently added the python support for cluster mode.

@tgravescs
Copy link
Contributor

Jenkins, test this please

@SparkQA
Copy link

SparkQA commented Apr 14, 2015

Test build #30248 has finished for PR 5490 at commit ed67797.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.
  • This patch does not change any dependencies.

@andrewor14
Copy link
Contributor

I actually don't think we should merge this. It's not specific to YARN that interactive shells cannot run in cluster mode. I agree that we should have a centralized table somewhere that details what applications are supported on what modes, but this doesn't seem like the right place.

@Sephiroth-Lin Sephiroth-Lin deleted the SPARK-5689 branch May 15, 2016 10:15
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.

5 participants