chengshiwen edited a comment on pull request #4739: URL: https://github.com/apache/incubator-dolphinscheduler/pull/4739#issuecomment-775725337
Please think about this as a regular user, not a developer. Why must the sql task be bound to mail? Why must the query result be sent by mail when executing the SELECT statement? Can the sql task be executed without mail? Both executing SQL and sending mail are independent. Even with the `Query` type, sending mail should be optional. But it is required now, it's reasonable and minimal cost to catch the `send mail failed` exception. The exception should not be thrown to the user, so the warning is just enough. ---------------------------------------------------------------- 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: [email protected]
