You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2020/10/24 16:49:18 UTC
[GitHub] [airflow] petedejoy opened a new pull request #11829: Add providers, operators, and hooks readmes
petedejoy opened a new pull request #11829:
URL: https://github.com/apache/airflow/pull/11829
It took me quite a while to piece together how exactly operators, hooks, and providers would be released and distributed in Airflow 2.0 and beyond from various mailing list threads, AIPs, and slack conversations. Hoping that adding a little bit of color to readmes in the relevant subdirectories will help users who may be less familiar with the maintenance of the project figure out how exactly the operators, hooks, and providers directories differ. Someone should give these a quick once-over to validate that my mental model is accurate.
---
Read the **[Pull Request Guidelines](https://github.com/apache/airflow/blob/master/CONTRIBUTING.rst#pull-request-guidelines)** for more information.
In case of fundamental code change, Airflow Improvement Proposal ([AIP](https://cwiki.apache.org/confluence/display/AIRFLOW/Airflow+Improvements+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 [UPDATING.md](https://github.com/apache/airflow/blob/master/UPDATING.md).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] ryw merged pull request #11829: Add providers, operators, and hooks readmes
Posted by GitBox <gi...@apache.org>.
ryw merged pull request #11829:
URL: https://github.com/apache/airflow/pull/11829
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] potiuk commented on pull request #11829: Add providers, operators, and hooks readmes
Posted by GitBox <gi...@apache.org>.
potiuk commented on pull request #11829:
URL: https://github.com/apache/airflow/pull/11829#issuecomment-716000061
Just one EOF static check failiing :)
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] petedejoy commented on pull request #11829: Add providers, operators, and hooks readmes
Posted by GitBox <gi...@apache.org>.
petedejoy commented on pull request #11829:
URL: https://github.com/apache/airflow/pull/11829#issuecomment-716000932
Thanks @potiuk! Is that failing because I need to add a new line at the end of these files?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] ryw commented on a change in pull request #11829: Add providers, operators, and hooks readmes
Posted by GitBox <gi...@apache.org>.
ryw commented on a change in pull request #11829:
URL: https://github.com/apache/airflow/pull/11829#discussion_r511528569
##########
File path: airflow/operators/README.md
##########
@@ -0,0 +1,24 @@
+<!--
+ 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.
+ -->
+
+# Airflow Operators
+
+Airflow operators are modules that represent a single, ideally idempotent task. They are arbiters of the logic that executes when your DAG runs.
Review comment:
I'd capitalize "Airflow Operators"
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] github-actions[bot] commented on pull request #11829: Add providers, operators, and hooks readmes
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #11829:
URL: https://github.com/apache/airflow/pull/11829#issuecomment-716004004
[The Workflow run](https://github.com/apache/airflow/actions/runs/326187966) is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks$,^Build docs$,^Spell check docs$,^Backport packages$,^Checks: Helm tests$,^Test OpenAPI*.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] github-actions[bot] commented on pull request #11829: Add providers, operators, and hooks readmes
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #11829:
URL: https://github.com/apache/airflow/pull/11829#issuecomment-716028444
[The Workflow run](https://github.com/apache/airflow/actions/runs/326215228) is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks$,^Build docs$,^Spell check docs$,^Backport packages$,^Checks: Helm tests$,^Test OpenAPI*.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org