[GitHub] [incubator-dolphinscheduler] chengshiwen opened a new pull request #4739: [Fix-4738][Mail] Fix send mail failed instead of execute sql error

[GitBox]

chengshiwen opened a new pull request #4739:
URL: https://github.com/apache/incubator-dolphinscheduler/pull/4739


   ## *Tips*
   - *Thanks very much for contributing to Apache DolphinScheduler.*
   - *Please review 
https://dolphinscheduler.apache.org/en-us/community/index.html before opening a 
pull request.*
   
   ## What is the purpose of the pull request
   
   **Fix send mail failed instead of execute sql error**
   
   This closes #4738 #4710 #4621
   
   ## Brief change log
   
   *(for example:)*
     - *Capture the `RuntimeException("send mail failed!")` in 
`server/worker/task/sql/SqlTask.java`*
   
   ## Verify this pull request
   
   This change added tests and can be verified as follows:
   
   *(example:)*
   
     - *Manually verified the change by testing locally.*
   
   After fix:
   
![image](https://user-images.githubusercontent.com/4902714/107212020-4fd96d80-6a41-11eb-93b9-2db1c9de9c0d.png)
   


----------------------------------------------------------------
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:
us...@infra.apache.org