New AWS sensor — DynamoDBValueSensor

[mrichman]

DynamoDBValueSensor waits for an attribute value to be present for an item in a DynamoDB table.

[boring-cyborg]

Congratulations on your first Pull Request and welcome to the Apache Airflow community! If you have any issues or are unsure about any anything please check our Contribution Guide (https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst)
Here are some useful points:

• Pay attention to the quality of your code (flake8, mypy and type annotations). Our pre-commits will help you with that.
• In case of a new feature add useful documentation (in docstrings or in docs/ directory). Adding a new operator? Check this short guide Consider adding an example DAG that shows how users should use it.
• Consider using Breeze environment for testing locally, it's a heavy docker but it ships with a working Airflow and a lot of integrations.
• Be patient and persistent. It might take some time to get a review or get the final approval from Committers.
• Please follow ASF Code of Conduct for all communication including (but not limited to) comments on Pull Requests, Mailing list and Slack.
• Be sure to read the Airflow Coding style.
  Apache Airflow is a community-driven project and together we are making it better 🚀.
  In case of doubts contact the developers at:
  Mailing List: dev@airflow.apache.org
  Slack: https://s.apache.org/airflow-slack

[mrichman]

@o-nikolas Please review at your convenience. Thanks.

[ferruzzi]

This is marked as a draft, do you want a review on it or are you still working on it?

[mrichman]

@ferruzzi Yes please review. @o-nikolas suggested I create as a draft to limit visibility. Thanks!

[o-nikolas]

Hey @mrichman,

Any plans to make more progress on this one?

[mrichman]

@o-nikolas Yes indeed, just haven't come up for air with all my new customers. It's definitely toward the top of my (ever growing) todo list.

[ferruzzi]

Left two non-blocking ideas. Once the tests pass, I'm happy to approve.

[mrichman]

@ferruzzi Am I to use tests/providers/amazon/aws/providers for my unit test?

[eladkal]

@ferruzzi Am I to use tests/providers/amazon/aws/providers for my unit test?

It should be in
tests/providers/amazon/aws/sensors/test_dynamodb.py

If file doesn't exist then create it.

[ferruzzi]

What Elad said 😄

Basically just replace the airflow in the file's path with tests so tests for airflow/providers/amazon/aws/sensors/test_dynamodb.py go in tests/providers/amazon/aws/sensors/test_dynamodb.py

[ferruzzi]

I think it looks good.

[o-nikolas]

lgtm, I just triggered another build, if it goes green I'll merge :)

[mrichman]

[o-nikolas]

Just approved another build on the latest changes! 🤞

[boring-cyborg]

Awesome work, congrats on your first merged pull request! You are invited to check our Issue Tracker for additional contributions.

[ferruzzi]

Nice, congrats!

[potiuk]

Indeed.

[o-nikolas]

congrats on the first contribution @mrichman!!