[GitHub] [spark] HeartSaVioR edited a comment on pull request #30037: [SPARK-33146][CORE] Check for non-fatal errors when loading new applications in SHS

[GitBox <gi...@apache.org>]

HeartSaVioR edited a comment on pull request #30037:
URL: https://github.com/apache/spark/pull/30037#issuecomment-708930507


   Thanks for catching the issue and sorry to miss the divergence between branches before.
   
   Looks like the root cause is missing commit in branch-3.0: SPARK-32557 was a kind of follow-up for SPARK-32529, but while SPARK-32529 was landed to both master and branch-3.0, SPARK-32557 was only landed to master.
   
   Instead of keeping unnecessary divergence between branches, I'd propose to port back SPARK-32557 to branch-3.0, and re-introduce this to branch-3.0. I expect no change will be needed after injecting SPARK-32557, but I'll do manual check. WDYT?


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org