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 2025/01/23 11:33:22 UTC

[PR] Move standad provider to the new structure [airflow]

potiuk opened a new pull request, #45964:
URL: https://github.com/apache/airflow/pull/45964

   This one also adds some missing steps in provider migration script. improves the output and instructions and make it more robust for various variations of provider features and usage of imports.
   
   With those changes `standard` provider was 100% moved automatically.
   
   <!--
    Licensed to the Apache Software Foundation (ASF) under one
    or more contributor license agreements.  See the NOTICE file
    distributed with this work for additional information
    regarding copyright ownership.  The ASF licenses this file
    to you under the Apache License, Version 2.0 (the
    "License"); you may not use this file except in compliance
    with the License.  You may obtain a copy of the License at
   
      http://www.apache.org/licenses/LICENSE-2.0
   
    Unless required by applicable law or agreed to in writing,
    software distributed under the License is distributed on an
    "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
    KIND, either express or implied.  See the License for the
    specific language governing permissions and limitations
    under the License.
    -->
   
   <!--
   Thank you for contributing! Please make sure that your code changes
   are covered with tests. And in case of new features or big changes
   remember to adjust the documentation.
   
   Feel free to ping committers for the review!
   
   In case of an existing issue, reference it using one of the following:
   
   closes: #ISSUE
   related: #ISSUE
   
   How to write a good git commit message:
   http://chris.beams.io/posts/git-commit/
   -->
   
   
   
   <!-- Please keep an empty line above the dashes. -->
   ---
   **^ Add meaningful description above**
   Read the **[Pull Request Guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#pull-request-guidelines)** for more information.
   In case of fundamental code changes, an Airflow Improvement Proposal ([AIP](https://cwiki.apache.org/confluence/display/AIRFLOW/Airflow+Improvement+Proposals)) is needed.
   In case of a new dependency, check compliance with the [ASF 3rd Party License Policy](https://www.apache.org/legal/resolved.html#category-x).
   In case of backwards incompatible changes please leave a note in a newsfragment file, named `{pr_number}.significant.rst` or `{issue_number}.significant.rst`, in [newsfragments](https://github.com/apache/airflow/tree/main/newsfragments).
   


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


Re: [PR] Move standard and common.sql provider to the new structure [airflow]

Posted by "dabla (via GitHub)" <gi...@apache.org>.
dabla commented on PR #45964:
URL: https://github.com/apache/airflow/pull/45964#issuecomment-2611825142

   @potiuk shouldn't we move the generic transfer already to the common sql provider here?


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


Re: [PR] Move standad provider to the new structure [airflow]

Posted by "gopidesupavan (via GitHub)" <gi...@apache.org>.
gopidesupavan commented on code in PR #45964:
URL: https://github.com/apache/airflow/pull/45964#discussion_r1926981690


##########
providers/standard/README.rst:
##########
@@ -0,0 +1,62 @@
+
+ .. Licensed to the Apache Software Foundation (ASF) under one
+    or more contributor license agreements.  See the NOTICE file
+    distributed with this work for additional information
+    regarding copyright ownership.  The ASF licenses this file
+    to you under the Apache License, Version 2.0 (the
+    "License"); you may not use this file except in compliance
+    with the License.  You may obtain a copy of the License at
+
+ ..   http://www.apache.org/licenses/LICENSE-2.0
+
+ .. Unless required by applicable law or agreed to in writing,
+    software distributed under the License is distributed on an
+    "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+    KIND, either express or implied.  See the License for the
+    specific language governing permissions and limitations
+    under the License.
+
+ .. NOTE! THIS FILE IS AUTOMATICALLY GENERATED AND WILL BE OVERWRITTEN!
+
+ .. IF YOU WANT TO MODIFY TEMPLATE FOR THIS FILE, YOU SHOULD MODIFY THE TEMPLATE
+    `PROVIDER_README_TEMPLATE.rst.jinja2` IN the `dev/breeze/src/airflow_breeze/templates` DIRECTORY
+
+
+Package ``apache-airflow-providers-standard``
+
+Release: ``0.0.3``
+
+
+Airflow Standard Provider
+
+
+Provider package
+----------------
+
+This is a provider package for ``standard`` provider. All classes for this provider package
+are in ``airflow.providers.standard`` python package.
+
+You can find package information and changelog for the provider
+in the `documentation <https://airflow.apache.org/docs/apache-airflow-providers-standard/0.0.3/>`_.
+
+Installation
+------------
+
+You can install this package on top of an existing Airflow 2 installation (see ``Requirements`` below
+for the minimum Airflow version supported) via
+``pip install apache-airflow-providers-standard``
+
+The package supports the following python versions: 3.9,3.10,3.11,3.12
+
+Requirements
+------------
+
+=======================================  ==================
+PIP package                              Version required
+=======================================  ==================
+``apache-airflow``                       ``>=2.9.0``
+``apache-airflow-providers-common-sql``  ``>=1.20.0``

Review Comment:
   Do we need common sql? believe common compat provider required.



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


Re: [PR] Move standard and common.sql provider to the new structure [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #45964:
URL: https://github.com/apache/airflow/pull/45964#issuecomment-2610864789

   > @potiuk so far I'm seeing the same failures
   > 
   > https://github.com/apache/airflow/actions/runs/12933644386/job/36073123320?pr=45974
   
   Ah - you had the same errors because you rebased on top of my in-progress move standard providers, but if you do it direcly on main now - it should work.


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


Re: [PR] Move standard, alibaba and common.sql provider to the new structure [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #45964:
URL: https://github.com/apache/airflow/pull/45964#issuecomment-2613844232

   > oh good catch, thanks Jarek, this move revealing good stuff. :)
   
   Yeah. I actually love to do these kinds of refactoring, they usually reval some `dead bodies in the closet` :D


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


Re: [PR] Move standard provider to the new structure [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #45964:
URL: https://github.com/apache/airflow/pull/45964#issuecomment-2610804387

   > @potiuk so far I'm seeing the same failures
   > 
   > https://github.com/apache/airflow/actions/runs/12933644386/job/36073123320?pr=45974
   
   On it. 


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


Re: [PR] Move standard and common.sql provider to the new structure [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #45964:
URL: https://github.com/apache/airflow/pull/45964#issuecomment-2613152659

   Hah... Found a few things missing from the previous moves - for example `task_sdk` was still present and installed in our compatibility tests - and started to mess around with pre-airflow-3 versions :).
   


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


Re: [PR] Move standard, alibaba and common.sql provider to the new structure [airflow]

Posted by "gopidesupavan (via GitHub)" <gi...@apache.org>.
gopidesupavan commented on PR #45964:
URL: https://github.com/apache/airflow/pull/45964#issuecomment-2613683722

   > Also found out that `decorators/test_python.py` have not been moved to standards provider when we moved stuff :).
   > 
   > This is actually nice thing about this splitting - we will likely find out all the stuff where providers are still imported from core tests :)
   
   ah i see, yes indeed. 


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


Re: [PR] Move standard, alibaba and common.sql provider to the new structure [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #45964:
URL: https://github.com/apache/airflow/pull/45964#issuecomment-2613250864

   I also accidentally added alibaba yesterday during demo :facepalm: ... But since it's already migrated, I will leave it in - updated description and title.


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


Re: [PR] Move standard provider to the new structure [airflow]

Posted by "dimberman (via GitHub)" <gi...@apache.org>.
dimberman commented on PR #45964:
URL: https://github.com/apache/airflow/pull/45964#issuecomment-2610423204

   @potiuk so far I'm seeing the same failures
   
   https://github.com/apache/airflow/actions/runs/12933644386/job/36073123320?pr=45974


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


Re: [PR] Move standard provider to the new structure [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #45964:
URL: https://github.com/apache/airflow/pull/45964#issuecomment-2610034175

   Also found out that `decorators/test_python.py` have not been moved to standards provider when we moved stuff :).
   
   This is actually nice thing about this splitting - we will likely find out all the stuff where providers are  still imported from core tests :)


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


Re: [PR] Move standard and common.sql provider to the new structure [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #45964:
URL: https://github.com/apache/airflow/pull/45964#issuecomment-2613004981

   > @potiuk shouldn't we move the generic transfer already to the common sql provider here?
   
   I wanted to avoid it to do too many things at the same time.


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


Re: [PR] Move standard provider to the new structure [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #45964:
URL: https://github.com/apache/airflow/pull/45964#issuecomment-2610816562

   OK. looks like I have to convert common.sql first - to make that one succeed


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


Re: [PR] Move standard, alibaba and common.sql provider to the new structure [airflow]

Posted by "gopidesupavan (via GitHub)" <gi...@apache.org>.
gopidesupavan commented on PR #45964:
URL: https://github.com/apache/airflow/pull/45964#issuecomment-2613683039

   > Hah... Found a few things missing from the previous moves - for example `task_sdk` was still present and installed in our compatibility tests - and started to mess around with pre-[airflow-3](https://issues.apache.org/jira/browse/AIRFLOW-3) versions :).
   
   oh good catch, thanks Jarek, this move revealing good stuff. :) 


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


Re: [PR] Move standard, alibaba and common.sql provider to the new structure [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #45964:
URL: https://github.com/apache/airflow/pull/45964#issuecomment-2614119350

   Finally got it green... This single PR allowed me to trace and fix quite a number of small issues we had with moving to the "providers" and now to the "newer providers" strucuture... Next step will be to launch full-blown migration process :).


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


Re: [PR] Move standard provider to the new structure [airflow]

Posted by "gopidesupavan (via GitHub)" <gi...@apache.org>.
gopidesupavan commented on code in PR #45964:
URL: https://github.com/apache/airflow/pull/45964#discussion_r1927113849


##########
providers/standard/README.rst:
##########
@@ -0,0 +1,62 @@
+
+ .. Licensed to the Apache Software Foundation (ASF) under one
+    or more contributor license agreements.  See the NOTICE file
+    distributed with this work for additional information
+    regarding copyright ownership.  The ASF licenses this file
+    to you under the Apache License, Version 2.0 (the
+    "License"); you may not use this file except in compliance
+    with the License.  You may obtain a copy of the License at
+
+ ..   http://www.apache.org/licenses/LICENSE-2.0
+
+ .. Unless required by applicable law or agreed to in writing,
+    software distributed under the License is distributed on an
+    "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+    KIND, either express or implied.  See the License for the
+    specific language governing permissions and limitations
+    under the License.
+
+ .. NOTE! THIS FILE IS AUTOMATICALLY GENERATED AND WILL BE OVERWRITTEN!
+
+ .. IF YOU WANT TO MODIFY TEMPLATE FOR THIS FILE, YOU SHOULD MODIFY THE TEMPLATE
+    `PROVIDER_README_TEMPLATE.rst.jinja2` IN the `dev/breeze/src/airflow_breeze/templates` DIRECTORY
+
+
+Package ``apache-airflow-providers-standard``
+
+Release: ``0.0.3``
+
+
+Airflow Standard Provider
+
+
+Provider package
+----------------
+
+This is a provider package for ``standard`` provider. All classes for this provider package
+are in ``airflow.providers.standard`` python package.
+
+You can find package information and changelog for the provider
+in the `documentation <https://airflow.apache.org/docs/apache-airflow-providers-standard/0.0.3/>`_.
+
+Installation
+------------
+
+You can install this package on top of an existing Airflow 2 installation (see ``Requirements`` below
+for the minimum Airflow version supported) via
+``pip install apache-airflow-providers-standard``
+
+The package supports the following python versions: 3.9,3.10,3.11,3.12
+
+Requirements
+------------
+
+=======================================  ==================
+PIP package                              Version required
+=======================================  ==================
+``apache-airflow``                       ``>=2.9.0``
+``apache-airflow-providers-common-sql``  ``>=1.20.0``

Review Comment:
   Ah okay :)



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


Re: [PR] Move standard and common.sql provider to the new structure [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #45964:
URL: https://github.com/apache/airflow/pull/45964#issuecomment-2610858496

   Seems that easiest way is to :crossed_fingers: migrate common.sql and standard together


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


Re: [PR] Move standard provider to the new structure [airflow]

Posted by "dimberman (via GitHub)" <gi...@apache.org>.
dimberman commented on PR #45964:
URL: https://github.com/apache/airflow/pull/45964#issuecomment-2610323406

   @potiuk I've created a PR here with a minimal k8s fix to ensure that the fix provided will fix our tests too. Thank you for responding quickly!
   
   https://github.com/apache/airflow/pull/45974


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


Re: [PR] Move standard, alibaba and common.sql provider to the new structure [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk merged PR #45964:
URL: https://github.com/apache/airflow/pull/45964


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


Re: [PR] Move standard provider to the new structure [airflow]

Posted by "dabla (via GitHub)" <gi...@apache.org>.
dabla commented on code in PR #45964:
URL: https://github.com/apache/airflow/pull/45964#discussion_r1927428571


##########
providers/standard/README.rst:
##########
@@ -0,0 +1,62 @@
+
+ .. Licensed to the Apache Software Foundation (ASF) under one
+    or more contributor license agreements.  See the NOTICE file
+    distributed with this work for additional information
+    regarding copyright ownership.  The ASF licenses this file
+    to you under the Apache License, Version 2.0 (the
+    "License"); you may not use this file except in compliance
+    with the License.  You may obtain a copy of the License at
+
+ ..   http://www.apache.org/licenses/LICENSE-2.0
+
+ .. Unless required by applicable law or agreed to in writing,
+    software distributed under the License is distributed on an
+    "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+    KIND, either express or implied.  See the License for the
+    specific language governing permissions and limitations
+    under the License.
+
+ .. NOTE! THIS FILE IS AUTOMATICALLY GENERATED AND WILL BE OVERWRITTEN!
+
+ .. IF YOU WANT TO MODIFY TEMPLATE FOR THIS FILE, YOU SHOULD MODIFY THE TEMPLATE
+    `PROVIDER_README_TEMPLATE.rst.jinja2` IN the `dev/breeze/src/airflow_breeze/templates` DIRECTORY
+
+
+Package ``apache-airflow-providers-standard``
+
+Release: ``0.0.3``
+
+
+Airflow Standard Provider
+
+
+Provider package
+----------------
+
+This is a provider package for ``standard`` provider. All classes for this provider package
+are in ``airflow.providers.standard`` python package.
+
+You can find package information and changelog for the provider
+in the `documentation <https://airflow.apache.org/docs/apache-airflow-providers-standard/0.0.3/>`_.
+
+Installation
+------------
+
+You can install this package on top of an existing Airflow 2 installation (see ``Requirements`` below
+for the minimum Airflow version supported) via
+``pip install apache-airflow-providers-standard``
+
+The package supports the following python versions: 3.9,3.10,3.11,3.12
+
+Requirements
+------------
+
+=======================================  ==================
+PIP package                              Version required
+=======================================  ==================
+``apache-airflow``                       ``>=2.9.0``
+``apache-airflow-providers-common-sql``  ``>=1.20.0``

Review Comment:
   > For now yes. We have "generic_transfer" as part of standard operator - and after some @dabla changes it needs common.sql in specific version. I don't think standard uses (nor should use) compat - because standard is almost exclusively USED by compat - not the other way round.
   > 
   > But yeah I have to come back to some @dabla changes (been busy with that one) - and we already agreed generic_transfer should be moved to common.sql - so once we do that, we will be able to remove common.sql from here.
   
   Normally after [my changes](https://github.com/apache/airflow/pull/44809/), the generic_transfer operator will be moved to common-sql provider, which would mean the standard provider won't need the common-sql dependency anymore.



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


Re: [PR] Move standard, alibaba and common.sql provider to the new structure [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #45964:
URL: https://github.com/apache/airflow/pull/45964#issuecomment-2613252396

   And added few more replacements (logo/ .pre-commit-config.yaml) that could also be automated easliy (standard provider did not have them - but alibaba did).


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


Re: [PR] Move standard provider to the new structure [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #45964:
URL: https://github.com/apache/airflow/pull/45964#issuecomment-2610101521

   And there are still two things to fix I see :)
   


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


Re: [PR] Move standard provider to the new structure [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on code in PR #45964:
URL: https://github.com/apache/airflow/pull/45964#discussion_r1927000267


##########
providers/standard/README.rst:
##########
@@ -0,0 +1,62 @@
+
+ .. Licensed to the Apache Software Foundation (ASF) under one
+    or more contributor license agreements.  See the NOTICE file
+    distributed with this work for additional information
+    regarding copyright ownership.  The ASF licenses this file
+    to you under the Apache License, Version 2.0 (the
+    "License"); you may not use this file except in compliance
+    with the License.  You may obtain a copy of the License at
+
+ ..   http://www.apache.org/licenses/LICENSE-2.0
+
+ .. Unless required by applicable law or agreed to in writing,
+    software distributed under the License is distributed on an
+    "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+    KIND, either express or implied.  See the License for the
+    specific language governing permissions and limitations
+    under the License.
+
+ .. NOTE! THIS FILE IS AUTOMATICALLY GENERATED AND WILL BE OVERWRITTEN!
+
+ .. IF YOU WANT TO MODIFY TEMPLATE FOR THIS FILE, YOU SHOULD MODIFY THE TEMPLATE
+    `PROVIDER_README_TEMPLATE.rst.jinja2` IN the `dev/breeze/src/airflow_breeze/templates` DIRECTORY
+
+
+Package ``apache-airflow-providers-standard``
+
+Release: ``0.0.3``
+
+
+Airflow Standard Provider
+
+
+Provider package
+----------------
+
+This is a provider package for ``standard`` provider. All classes for this provider package
+are in ``airflow.providers.standard`` python package.
+
+You can find package information and changelog for the provider
+in the `documentation <https://airflow.apache.org/docs/apache-airflow-providers-standard/0.0.3/>`_.
+
+Installation
+------------
+
+You can install this package on top of an existing Airflow 2 installation (see ``Requirements`` below
+for the minimum Airflow version supported) via
+``pip install apache-airflow-providers-standard``
+
+The package supports the following python versions: 3.9,3.10,3.11,3.12
+
+Requirements
+------------
+
+=======================================  ==================
+PIP package                              Version required
+=======================================  ==================
+``apache-airflow``                       ``>=2.9.0``
+``apache-airflow-providers-common-sql``  ``>=1.20.0``

Review Comment:
   For now yes. We have "generic_transfer" as part of standard operator - and after some @dabla changes it needs common.sql in specific version. I don't think standard uses (nor should use) compat - because standard is almost exclusively USED by compat - not the other way round. 
   
   But yeah I have to come back to some @dabla changes (been busy with that one) - and we already agreed generic_transfer should be moved to common.sql - so once we do that, we will be able to remove common.sql from here.



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


Re: [PR] Move standard provider to the new structure [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #45964:
URL: https://github.com/apache/airflow/pull/45964#issuecomment-2610024713

   Got some fixes in :crossed_fingers: . also found a few small things - now IDE integration works nicely with the providers in separate folders (it was broken for a few days for the new providers).


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


Re: [PR] Move standard and common.sql provider to the new structure [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #45964:
URL: https://github.com/apache/airflow/pull/45964#issuecomment-2610922769

   Some more fun to check - but at least the image builds now. I am glad I decided to do it slowly and incrementally ;) 


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


Re: [PR] Move standard, alibaba and common.sql provider to the new structure [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #45964:
URL: https://github.com/apache/airflow/pull/45964#issuecomment-2613922104

   And we need to add `task-sdk` at least for now - until we release it in PyPI - in order to run various provider verification tests as those are run with packages built and installed from sources :).
   
   Another step closer to get to the package split - @ashb


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