Technoboy- opened a new pull request #1424: simply server module configs
URL: https://github.com/apache/incubator-dolphinscheduler/pull/1424
 
 
   this pr is for #1423 .
   for reviewers, it's easy to review this pr in below way.
   1. this pr will delete master and worker properties file to simply server 
configs files. Using WorkerConfig and ServerConfig as the default value for old 
configs.  so user wanna change the value , just config it in respect 
application-xxx.properties.
   2. because delete Constants fields, so all the classes using it's references 
will easy to find.
   3. so reviewers can pull first, and if build pass, try to run several 
different tasks for testing.
   4. for this pr modifies about 16 files, it's hard to write UT.
   5. thanks for reviewers and regards.
   
   

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


With regards,
Apache Git Services

Reply via email to