743294668 commented on pull request #2832:
URL: 
https://github.com/apache/incubator-dolphinscheduler/pull/2832#issuecomment-639289308


   > It's better not to change the scope of appendDatabase. You can override 
the getJdbcUrl directly
   
   At first I did getJdbcUrl modified, and then I noticed #2812 modification, I 
made changes based on this basis, and covered appendDatabase.
   For now, do you want to rewrite getJdbcUrl instead? Any other suggestions? 
Looking forward for your response, thank you


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


Reply via email to