[GitHub] [shardingsphere-elastic-job-lite] Technoboy- opened a new issue #782: Refactor MonitorService

[GitBox <gi...@apache.org>]

Technoboy- opened a new issue #782:
URL: https://github.com/apache/shardingsphere-elastic-job-lite/issues/782


   ## Feature Request
   Currently, if a job has something wrong, and need to dump data, we have to configure a monitor port per job. As the job increases, the port grows. I think monitor service is an useful function that should support by default. And every project, we should only config one monitor service and dump with command 'dump jobName' is much more better.  So I would like to talk about the refactoring about MonitorService.
   1. We need to add monitor config namespace as like 'reg' and 'job'.
   2. Monitor config relies on reg due to current implementation, so still keep it.
   3. Monitor config should have a default port and be enabled by default.
   4. Refactor the codes that involves with MonitorService.
   And please give some more advise.
    


----------------------------------------------------------------
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-elastic-job-lite] Technoboy- commented on issue #782: Refactor MonitorService

[GitBox <gi...@apache.org>]

Technoboy- commented on issue #782:
URL: https://github.com/apache/shardingsphere-elastic-job-lite/issues/782#issuecomment-645131831


   @terrymanu 


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