This is an automated email from the ASF dual-hosted git repository.
dongjoon pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/master by this push:
new 4b0ceaddde17 [SPARK-46664][CORE][TESTS][FOLLOWUP] Add a persistent
`DriverInfo`
4b0ceaddde17 is described below
commit 4b0ceaddde17a870289f20269526fba39242bbda
Author: Dongjoon Hyun <[email protected]>
AuthorDate: Thu Jan 11 08:34:03 2024 -0800
[SPARK-46664][CORE][TESTS][FOLLOWUP] Add a persistent `DriverInfo`
### What changes were proposed in this pull request?
This is a follow-up of #44673 to fix the test case by adding a persistent
`DriverInfo`.
### Why are the changes needed?
To make the test case fail without the main patch. Previously, this is
missed. So, the test case succeeds without the main patch.
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
Pass the CIs.
### Was this patch authored or co-authored using generative AI tooling?
No.
Closes #44693 from dongjoon-hyun/SPARK-46664-2.
Authored-by: Dongjoon Hyun <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
---
.../scala/org/apache/spark/deploy/master/MasterSuite.scala | 12 ++++++++++++
1 file changed, 12 insertions(+)
diff --git
a/core/src/test/scala/org/apache/spark/deploy/master/MasterSuite.scala
b/core/src/test/scala/org/apache/spark/deploy/master/MasterSuite.scala
index 6490ee3e8241..e7146747b843 100644
--- a/core/src/test/scala/org/apache/spark/deploy/master/MasterSuite.scala
+++ b/core/src/test/scala/org/apache/spark/deploy/master/MasterSuite.scala
@@ -256,6 +256,18 @@ class MasterSuite extends SparkFunSuite
conf.set(RECOVERY_MODE_FACTORY,
classOf[FakeRecoveryModeFactory].getCanonicalName)
conf.set(MASTER_REST_SERVER_ENABLED, false)
+ val fakeDriverInfo = new DriverInfo(
+ startTime = 0,
+ id = "test_driver",
+ desc = new DriverDescription(
+ jarUrl = "",
+ mem = 1024,
+ cores = 1,
+ supervise = false,
+ command = new Command("", Nil, Map.empty, Nil, Nil, Nil)),
+ submitDate = new Date())
+ FakeRecoveryModeFactory.persistentData.put(s"driver_${fakeDriverInfo.id}",
fakeDriverInfo)
+
var master: Master = null
try {
master = makeMaster(conf)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
