o-nikolas commented on code in PR #73506:
URL: https://github.com/apache/airflow/pull/73506#discussion_r4074272852


##########
providers/amazon/docs/changelog.rst:
##########
@@ -35,6 +38,58 @@ Changelog
   botocore configuration, which previously never reached it. Set these 
explicitly on the operator
   if the deferred half needs to differ from the synchronous half.
 
+Breaking changes
+~~~~~~~~~~~~~~~~
+
+* ``Build deferred AWS hooks from the operator's own settings (#72171)``

Review Comment:
   We chatted about this with the AWS team, we consider this a bug fix and not 
requiring a major version change. CC @vincbeck @ferruzzi 



##########
providers/duckdb/README.rst:
##########
@@ -57,28 +60,5 @@ PIP package                              Version required
 ``duckdb``                               ``>=1.2.0``
 =======================================  ==================
 
-Cross provider package dependencies
------------------------------------
-
-Those are dependencies that might be needed in order to use all the features 
of the package.
-You need to install the specified providers in order to use them.
-
-You can install such cross-provider dependencies when installing from PyPI. 
For example:
-
-.. code-block:: bash
-
-    pip install apache-airflow-providers-duckdb[common.sql]
-
-
-============================================================================================================
  ==============
-Dependent package                                                              
                               Extra
-============================================================================================================
  ==============
-`apache-airflow-providers-common-sql 
<https://airflow.apache.org/docs/apache-airflow-providers-common-sql>`_  
``common.sql``
-============================================================================================================
  ==============
-
-DuckDB version compatibility

Review Comment:
   > (duckdb isn't in this wave, so none of this reaches a published artifact — 
repo only.)
   
   Why would DuckDB not be in this wave @potiuk? We should definitely release 
it this wave, it is in a ready state (CC @shahar1)



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