Github user gatorsmile commented on a diff in the pull request:

    https://github.com/apache/spark/pull/18887#discussion_r139346016
  
    --- Diff: 
core/src/test/scala/org/apache/spark/deploy/history/HistoryServerSuite.scala ---
    @@ -74,6 +76,7 @@ class HistoryServerSuite extends SparkFunSuite with 
BeforeAndAfter with Matchers
           .set("spark.history.fs.logDirectory", logDir)
           .set("spark.history.fs.update.interval", "0")
           .set("spark.testing", "true")
    +      .set(LOCAL_STORE_DIR, storeDir.getAbsolutePath())
    --- End diff --
    
    This is another major history server test suite, which does not cover 
`InMemoryStore`. Since `InMemoryStore ` is by default, we might need a better 
test coverage.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to