LuciferYang commented on code in PR #41105:
URL: https://github.com/apache/spark/pull/41105#discussion_r1190842120
##########
core/src/test/scala/org/apache/spark/deploy/history/ApplicationCacheSuite.scala:
##########
@@ -381,4 +385,38 @@ class ApplicationCacheSuite extends SparkFunSuite with
MockitoSugar with Matcher
verify(resp).sendRedirect("http://localhost:18080/history/local-123/jobs/job/?id=2";)
}
+ test("Load new SparkUI during old one is detaching") {
Review Comment:
The new case should named as `SPARK-43403: ... xxx`.
And I think it would be best if the new case could be reproduced
`java.lang.IllegalStateException: DB is closed` without change of
`ApplicationCache` and pass with this pr.
If it is not easy to reproduce `DB is closed` (without change of
ApplicationCache) in this one, I think you can add other `test`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
