Skip to content

Conversation

@ferruzzi
Copy link
Contributor

@ferruzzi ferruzzi commented May 2, 2022

Cleans up the existing sample DAG and docs for the Athena operator and sensor.
Part of a project to standardize AWS sample dags and docs in preparation for adding System Testing.

Related: #21523
Related: #21475
Related: #21828
Related: #21920
etc...

@ferruzzi
Copy link
Contributor Author

ferruzzi commented May 2, 2022

@eladkal may want eyes on this one since he has reviewed most.all of the others in this collection of changes.

@ferruzzi ferruzzi force-pushed the ferruzzi/docs-update/athena branch from 7be17d2 to 661b381 Compare May 4, 2022 16:26
@ferruzzi
Copy link
Contributor Author

ferruzzi commented May 4, 2022

@josh-fell Made the suggested changes and rebased while I was at it. Thanks for the review.

@github-actions github-actions bot added the okay to merge It's ok to merge this PR as it does not require more tests label May 6, 2022
@github-actions
Copy link

github-actions bot commented May 6, 2022

The PR is likely OK to be merged with just subset of tests for default Python and Database versions without running the full matrix of tests, because it does not modify the core of Airflow. If the committers decide that the full tests matrix is needed, they will add the label 'full tests needed'. Then you should rebase to the latest main or amend the last commit of the PR, and push it with --force-with-lease.

@eladkal eladkal merged commit 46af5ba into apache:main May 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:providers kind:documentation okay to merge It's ok to merge this PR as it does not require more tests provider:amazon AWS/Amazon - related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants