[3/4] git commit: [SPARK-4166][Core] Add a backward compatibility test for ExecutorLostFailure
[SPARK-4166][Core] Add a backward compatibility test for ExecutorLostFailure Author: zsxwing Closes #3085 from zsxwing/SPARK-4166-back-comp and squashes the following commits: 89329f4 [zsxwing] Add a backward compatibility test for ExecutorLostFailure Project: http://git-wip-us.apache.org/repos/asf/spark/repo Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/68be37b8 Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/68be37b8 Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/68be37b8 Branch: refs/heads/branch-1.2 Commit: 68be37b823516dbeda066776bb060bf894db4e95 Parents: e0a043b Author: zsxwing Authored: Mon Nov 3 22:47:45 2014 -0800 Committer: Patrick Wendell Committed: Wed Nov 5 14:46:54 2014 -0800 -- .../scala/org/apache/spark/util/JsonProtocolSuite.scala | 9 + 1 file changed, 9 insertions(+) -- http://git-wip-us.apache.org/repos/asf/spark/blob/68be37b8/core/src/test/scala/org/apache/spark/util/JsonProtocolSuite.scala -- diff --git a/core/src/test/scala/org/apache/spark/util/JsonProtocolSuite.scala b/core/src/test/scala/org/apache/spark/util/JsonProtocolSuite.scala index 0103012..aec1e40 100644 --- a/core/src/test/scala/org/apache/spark/util/JsonProtocolSuite.scala +++ b/core/src/test/scala/org/apache/spark/util/JsonProtocolSuite.scala @@ -196,6 +196,15 @@ class JsonProtocolSuite extends FunSuite { assert(applicationStart === JsonProtocol.applicationStartFromJson(oldEvent)) } + test("ExecutorLostFailure backward compatibility") { +// ExecutorLostFailure in Spark 1.1.0 does not have an "Executor ID" property. +val executorLostFailure = ExecutorLostFailure("100") +val oldEvent = JsonProtocol.taskEndReasonToJson(executorLostFailure) + .removeField({ _._1 == "Executor ID" }) +val expectedExecutorLostFailure = ExecutorLostFailure("Unknown") +assert(expectedExecutorLostFailure === JsonProtocol.taskEndReasonFromJson(oldEvent)) + } + /** -- * | Helper test running methods | * --- */ - To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org For additional commands, e-mail: commits-h...@spark.apache.org
git commit: [SPARK-4166][Core] Add a backward compatibility test for ExecutorLostFailure
Repository: spark Updated Branches: refs/heads/master 9bdc8412a -> b671ce047 [SPARK-4166][Core] Add a backward compatibility test for ExecutorLostFailure Author: zsxwing Closes #3085 from zsxwing/SPARK-4166-back-comp and squashes the following commits: 89329f4 [zsxwing] Add a backward compatibility test for ExecutorLostFailure Project: http://git-wip-us.apache.org/repos/asf/spark/repo Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/b671ce04 Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/b671ce04 Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/b671ce04 Branch: refs/heads/master Commit: b671ce047d036b8923007902826038b01e836e8a Parents: 9bdc841 Author: zsxwing Authored: Mon Nov 3 22:47:45 2014 -0800 Committer: Andrew Or Committed: Mon Nov 3 22:47:45 2014 -0800 -- .../scala/org/apache/spark/util/JsonProtocolSuite.scala | 9 + 1 file changed, 9 insertions(+) -- http://git-wip-us.apache.org/repos/asf/spark/blob/b671ce04/core/src/test/scala/org/apache/spark/util/JsonProtocolSuite.scala -- diff --git a/core/src/test/scala/org/apache/spark/util/JsonProtocolSuite.scala b/core/src/test/scala/org/apache/spark/util/JsonProtocolSuite.scala index 0103012..aec1e40 100644 --- a/core/src/test/scala/org/apache/spark/util/JsonProtocolSuite.scala +++ b/core/src/test/scala/org/apache/spark/util/JsonProtocolSuite.scala @@ -196,6 +196,15 @@ class JsonProtocolSuite extends FunSuite { assert(applicationStart === JsonProtocol.applicationStartFromJson(oldEvent)) } + test("ExecutorLostFailure backward compatibility") { +// ExecutorLostFailure in Spark 1.1.0 does not have an "Executor ID" property. +val executorLostFailure = ExecutorLostFailure("100") +val oldEvent = JsonProtocol.taskEndReasonToJson(executorLostFailure) + .removeField({ _._1 == "Executor ID" }) +val expectedExecutorLostFailure = ExecutorLostFailure("Unknown") +assert(expectedExecutorLostFailure === JsonProtocol.taskEndReasonFromJson(oldEvent)) + } + /** -- * | Helper test running methods | * --- */ - To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org For additional commands, e-mail: commits-h...@spark.apache.org