You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "potiuk (via GitHub)" <gi...@apache.org> on 2023/04/24 09:49:52 UTC

[GitHub] [airflow] potiuk commented on pull request #30713: openlineage: add extractors for python and bash operators

potiuk commented on PR #30713:
URL: https://github.com/apache/airflow/pull/30713#issuecomment-1519772308

   > Hey @potiuk - as I wrote [in the implementation doc](https://docs.google.com/document/d/1YQMg4xePhetyjXCBHXwU7jbJpi5TEcqhwN8YbQ32P4Q) we want to migrate them, as we already have them it in OL. Then move to gradually reimplementing this in-operator. This would allow us to release `apache-airflow-provider-openlineage` with parity to `openlineage-airflow`, so that users moving to Airflow 2.6 could migrate seamlessly.
   
   Ah. OK makes sense. Can we mark all of those classes as :meta private: (not sure if this is the right way to mark the whole modules as private) and add description in those modules/classes so that it is clear those are private parts and not belonging to the API of the provider ?


-- 
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: commits-unsubscribe@airflow.apache.org

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