eschutho commented on a change in pull request #14208:
URL: https://github.com/apache/superset/pull/14208#discussion_r617062473
##########
File path: superset/databases/api.py
##########
@@ -839,3 +841,67 @@ def function_names(self, pk: int) -> Response:
if not database:
return self.response_404()
return self.response(200, function_names=database.function_names,)
+
+ @expose("/available/", methods=["GET"])
Review comment:
My initial thought is that this endpoint refers to a status of databases
and not a new resource. Should we make this endpoint more restful and if so,
how about something like GET /databases?status=available. On the other hand, I
realize that we already have a lot of other existing routes that aren't
particularly restful either, so then this route would be an outlier.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
