LuciferYang commented on code in PR #36119:
URL: https://github.com/apache/spark/pull/36119#discussion_r846569419


##########
core/src/test/scala/org/apache/spark/deploy/history/FsHistoryProviderSuite.scala:
##########
@@ -1261,74 +1328,6 @@ class FsHistoryProviderSuite extends SparkFunSuite with 
Matchers with Logging {
     }
   }
 
-  test("backwards compatibility with LogInfo from Spark 2.4") {

Review Comment:
   The 7 UT moved to `NoDiskBackendFsHistoryProviderSuite ` include:
   
   - `Parse application logs (inMemory = true)`
   - `backwards compatibility with LogInfo from Spark 2.4`
   - `SPARK-29755 LogInfo should be serialized/deserialized by jackson properly`
   - `SPARK-29755 AttemptInfoWrapper should be serialized/deserialized by 
jackson properly`
   - `SPARK-33146: don't let one bad rolling log folder prevent loading other 
applications`
   - `SPARK-36354: EventLogFileReader should skip rolling event log directories 
with no logs`



-- 
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]

Reply via email to