[GitHub] [incubator-kyuubi] lightning-L commented on a diff in pull request #3689: [KYUUBI #3688] add rest_api and kyuubi-admin docs for admin delete/list engine operation

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

lightning-L commented on code in PR #3689:
URL: https://github.com/apache/incubator-kyuubi/pull/3689#discussion_r1004106173


##########
docs/client/rest/rest_api.md:
##########
@@ -349,6 +349,43 @@ Gets the local log lines from this batch.
 | logRowSet | The log lines     | List of sting |
 | rowCount  | The log row count | Int           |
 
+## Admin Resource
+
+### POST /admin/refresh/hadoop_conf
+
+Refresh the Kyuubi server hadoop conf.
+
+### DELETE /admin/engine
+
+Delete specified engine.

Review Comment:
   yes, delete operation will delete an engine at a time. When the parameter is not specified, we will read the value from Kyuubi conf. For subdomain, if it is empty, we will use "default". Then we could construct the zk path.



-- 
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: notifications-unsubscribe@kyuubi.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org