bolkedebruin commented on a change in pull request #7138: [AIRFLOW-5912] Expose
lineage API
URL: https://github.com/apache/airflow/pull/7138#discussion_r368222650
##########
File path: airflow/api/client/api_client.py
##########
@@ -70,3 +70,12 @@ def delete_pool(self, name):
:param name: pool name
"""
raise NotImplementedError()
+
+ def get_lineage(self, dag_id: str, execution_date: str):
Review comment:
The class is intended for external use (I'm the original author). As
mentioned the local_client is a security problem and only exists for backwards
compatibility. The REST API can be used directly and the json_client provides
you a way to do this in any python application which is a provided
implementation of the REST API's interface rather than needing to create your
own client.
The CLI should switch to the `json_client` or, in the future, `grpc` maybe.
The local_client should be deprecated asap cause there is no way to do proper
authn/authz . Basically if you can run the command line client you have full
control over Airflow operationally.
You can argue that the client stuff could be moved to a semantic better
location but that will not be `cli`.
----------------------------------------------------------------
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
