[SPARK-3293] yarn's web show "SUCCEEDED" when the driver throw a exception in yarn-client

I think this could cause problems with YARN retry logic.  We shouldn't be finishing (unregistering with RM) unless we've explicitly succeeded or failed and don't want a retry to happen in those weird cases.  Since this is in shutdown hook I'm not sure we explicitly know that. 

i guess it comes down to us defining when the spark app is cleanly exiting.  This could be failed, killed, or succeeded.  Unfortunately I'm not sure spark really has this final reporting.