You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@linkis.apache.org by GitBox <gi...@apache.org> on 2022/03/23 10:02:12 UTC

[GitHub] [incubator-linkis] Fuu3214 opened a new issue #1781: [Bug] [engineConn] ECTaskEntranceMonitorService should be configurable to shut down

Fuu3214 opened a new issue #1781:
URL: https://github.com/apache/incubator-linkis/issues/1781


   ### Search before asking
   
   - [X] I searched the [issues](https://github.com/apache/incubator-linkis/issues) and found no similar issues.
   
   
   ### Linkis Component
   
   linkis-cg-engineConnManager
   
   ### What happened + What you expected to happen
   
   Currently ECTaskEntranceMonitor can be configured to shutdown in its start() function. But ECTaskEntranceMonitorService is always loaded and registered in listenerbus. So there can be situations that while the monitor and some of its deamon threads is off, the service can still register/unregister tasks to it.
   Also ECTaskEntranceMonitor is a functional module, wether shutdown or not should be decided by the one who calls it rather than inside the module.
   
   ### Relevent platform
   
   ——
   
   ### Reproduction script
   
   set "linkis.upstream.monitor.ectask.should.start=false" and submit any job
   
   ### Anything else
   
   _No response_
   
   ### Are you willing to submit a PR?
   
   - [X] Yes I am willing to submit a PR!


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

To unsubscribe, e-mail: dev-unsubscribe@linkis.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@linkis.apache.org
For additional commands, e-mail: dev-help@linkis.apache.org


[GitHub] [incubator-linkis] Davidhua1996 closed issue #1781: [Bug] [engineConn] ECTaskEntranceMonitorService should be configurable to shut down

Posted by GitBox <gi...@apache.org>.
Davidhua1996 closed issue #1781:
URL: https://github.com/apache/incubator-linkis/issues/1781


   


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

To unsubscribe, e-mail: dev-unsubscribe@linkis.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@linkis.apache.org
For additional commands, e-mail: dev-help@linkis.apache.org