eladkal commented on code in PR #31398:
URL: https://github.com/apache/airflow/pull/31398#discussion_r1248782685


##########
generated/provider_dependencies.json:
##########
@@ -231,7 +231,9 @@
     "deps": [
       "sqlparse>=0.4.2"
     ],
-    "cross-providers-deps": []
+    "cross-providers-deps": [
+      "openlineage"

Review Comment:
   We often have cases where providers co-depended. In such senario we release 
the providers in the same wave so users gain the functionality together.
   
   So this PR should have been put on hold till OL provider is ready then merge 
them in a sequence so both are ready for the same release wave. In providers 
release cycle we release from main branch we do not cherry pick commits thus 
anything merged will be released.
   
   
   Maybe @potiuk have another idea?



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