Skip to content

Conversation

uncleGen
Copy link
Contributor

... different storageLevels to a RDD

@SparkQA
Copy link

SparkQA commented Sep 22, 2014

QA tests have started for PR 2488 at commit 0e3831c.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Sep 22, 2014

QA tests have finished for PR 2488 at commit 0e3831c.

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

@pwendell
Copy link
Contributor

On this one, I would prefer the current approach of failing loudly rather than to silently continue. Existing users might also be depending on this behavior (e.g. they guard this in a try-catch) and for them this would break their code. Users who want to ignore this error can use a try/catch or a Try block in scala.

@uncleGen
Copy link
Contributor Author

@pwendell ah, make sense, I will close this PR. Thank you!

@uncleGen uncleGen closed this Sep 23, 2014
@uncleGen uncleGen deleted the master-minorfix branch March 24, 2015 02:35
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