EricGao888 commented on code in PR #9327:
URL: https://github.com/apache/dolphinscheduler/pull/9327#discussion_r845692867


##########
dolphinscheduler-common/src/main/resources/common.properties:
##########
@@ -91,3 +91,5 @@ development.state=false
 # rpc port
 alert.rpc.port=50052
 
+# Url endpoint for zeppelin RESTful API
+zeppelin.rest.url="http://localhost:8080";

Review Comment:
   > Maybe we should add a new component named `connection`, like Airflow, for 
the dynamic config task plugin. And in this way, we could not only have task 
config but also can change config without restart our server
   
   The `connection` idea sounds cool. It will be more convenient for users to 
change their credentials.



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to