Repository: spark
Updated Branches:
  refs/heads/master 9bdc8412a -> b671ce047


[SPARK-4166][Core] Add a backward compatibility test for ExecutorLostFailure

Author: zsxwing <[email protected]>

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 <[email protected]>
Authored: Mon Nov 3 22:47:45 2014 -0800
Committer: Andrew Or <[email protected]>
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: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to