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/08/05 11:19:54 UTC

[GitHub] [shardingsphere-elasticjob] BoulCheng opened a new pull request #1335: misfire depends on monitorExecution

BoulCheng opened a new pull request #1335:
URL: https://github.com/apache/shardingsphere-elasticjob/pull/1335


   misfire depends on monitorExecution. If monitorExecution flag is false then misfire will fail even if misfire flag is true, because a job running node can only be created if monitorExecution is marked true.
   
   


----------------------------------------------------------------
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] BoulCheng commented on pull request #1335: misfire depends on monitorExecution

Posted by GitBox <gi...@apache.org>.
BoulCheng commented on pull request #1335:
URL: https://github.com/apache/shardingsphere-elasticjob/pull/1335#issuecomment-669746311


   > It is better to do not change user configuration, it will make user confuse.
   > We can add the hint in to document
   
   OK


----------------------------------------------------------------
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] terrymanu commented on pull request #1335: misfire depends on monitorExecution

Posted by GitBox <gi...@apache.org>.
terrymanu commented on pull request #1335:
URL: https://github.com/apache/shardingsphere-elasticjob/pull/1335#issuecomment-669663384


   I update the document at #1336
   And this pr is wrong because of misfire is no relationship with execution monitor.


----------------------------------------------------------------
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] BoulCheng closed pull request #1335: misfire depends on monitorExecution

Posted by GitBox <gi...@apache.org>.
BoulCheng closed pull request #1335:
URL: https://github.com/apache/shardingsphere-elasticjob/pull/1335


   


----------------------------------------------------------------
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] terrymanu edited a comment on pull request #1335: misfire depends on monitorExecution

Posted by GitBox <gi...@apache.org>.
terrymanu edited a comment on pull request #1335:
URL: https://github.com/apache/shardingsphere-elasticjob/pull/1335#issuecomment-669663384


   I update the document at #1336.
   And this pr is wrong because of misfire is no relationship with execution monitor.


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