kacpermuda commented on code in PR #37620:
URL: https://github.com/apache/airflow/pull/37620#discussion_r1504503846


##########
docs/apache-airflow-providers-openlineage/guides/developer.rst:
##########
@@ -23,6 +23,42 @@
 Implementing OpenLineage in Operators

Review Comment:
   I agree, it's not short, but I think it's still the best way of presenting 
this information instead of splitting it into separate documents. There are 
three main parts to this page: three methods of lineage extraction, plus some 
additional information like how to contribute, troubleshoot, etc. I believe 
it's important to keep this information together, but I am open to discussion. 
Nevertheless, I think even if we decide to split it, we can create another PR 
and merge this one for now.



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