You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@shardingsphere.apache.org by GitBox <gi...@apache.org> on 2020/09/15 15:07:28 UTC

[GitHub] [shardingsphere-elasticjob] Technoboy- opened a new issue #1459: Add MessageJobErrorHandler to support sending message.

Technoboy- opened a new issue #1459:
URL: https://github.com/apache/shardingsphere-elasticjob/issues/1459


   To support for sending the messages when the job executes with errors, we decide to do some extensions for JobErrorHandler.
   - [ ] create a new module - elasticjob-extension/elasticjob-handler-email
   - [ ] load config from System parameters.
   - [ ] load config from specified conf.


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



[GitHub] [shardingsphere-elasticjob] Technoboy- commented on issue #1459: Add MessageJobErrorHandler to support sending message.

Posted by GitBox <gi...@apache.org>.
Technoboy- commented on issue #1459:
URL: https://github.com/apache/shardingsphere-elasticjob/issues/1459#issuecomment-697080734


   Ok, close first.


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



[GitHub] [shardingsphere-elasticjob] Technoboy- closed issue #1459: Add MessageJobErrorHandler to support sending message.

Posted by GitBox <gi...@apache.org>.
Technoboy- closed issue #1459:
URL: https://github.com/apache/shardingsphere-elasticjob/issues/1459


   


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



[GitHub] [shardingsphere-elasticjob] Technoboy- commented on issue #1459: Add MessageJobErrorHandler to support sending message.

Posted by GitBox <gi...@apache.org>.
Technoboy- commented on issue #1459:
URL: https://github.com/apache/shardingsphere-elasticjob/issues/1459#issuecomment-697080734


   Ok, close first.


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



[GitHub] [shardingsphere-elasticjob] viviel commented on issue #1459: Add MessageJobErrorHandler to support sending message.

Posted by GitBox <gi...@apache.org>.
viviel commented on issue #1459:
URL: https://github.com/apache/shardingsphere-elasticjob/issues/1459#issuecomment-692779978


   I can try it, please assign to me


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



[GitHub] [shardingsphere-elasticjob] viviel commented on issue #1459: Add MessageJobErrorHandler to support sending message.

Posted by GitBox <gi...@apache.org>.
viviel commented on issue #1459:
URL: https://github.com/apache/shardingsphere-elasticjob/issues/1459#issuecomment-695959107


   I thought about it carefully and found that this function didn't need to be done. Because each company may use different SMS platforms, and each SMS platform has its own configuration mechanism. 
   
   For example, some provide the rest interface or  the dependency package. users usually use the dependency rather than configure the rest interface themselves. If using the rest interface, the header parameters and URL parameters of each SMS platform may be different. In this way, it is not easy to map this information to the configuration file.
   
   So I want to see if you have any good suggestions
   
   @Technoboy- @terrymanu @TeslaCN 


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



[GitHub] [shardingsphere-elasticjob] Technoboy- closed issue #1459: Add MessageJobErrorHandler to support sending message.

Posted by GitBox <gi...@apache.org>.
Technoboy- closed issue #1459:
URL: https://github.com/apache/shardingsphere-elasticjob/issues/1459


   


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



[GitHub] [shardingsphere-elasticjob] viviel commented on issue #1459: Add MessageJobErrorHandler to support sending message.

Posted by GitBox <gi...@apache.org>.
viviel commented on issue #1459:
URL: https://github.com/apache/shardingsphere-elasticjob/issues/1459#issuecomment-695959107


   I thought about it carefully and found that this function didn't need to be done. Because each company may use different SMS platforms, and each SMS platform has its own configuration mechanism. 
   
   For example, some provide the rest interface or  the dependency package. users usually use the dependency rather than configure the rest interface themselves. If using the rest interface, the header parameters and URL parameters of each SMS platform may be different. In this way, it is not easy to map this information to the configuration file.
   
   So I want to see if you have any good suggestions
   
   @Technoboy- @terrymanu @TeslaCN 


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