Adding functionality to return export ARN for point in time export in DynamoDBToS3Operator

[tssahota]

Enable DynamoDbToS3Operator to return point in time export ARN to x_com to be used by downstream tasks

^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
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.

[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 Contributors' Guide (https://github.com/apache/airflow/blob/main/contributing-docs/README.rst)
Here are some useful points:

• Pay attention to the quality of your code (ruff, 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.
• Always keep your Pull Requests rebased, otherwise your build might fail due to changes not related to your commits.
  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

[vincbeck]

Could be great to have a unit test to cover this as well

[eladkal]

if adding return statement then there is no need for else

[tssahota]

Hi @eladkal, sorry for the delay, first time contributor, I assume you mean to remove the else and return the _export_entire_data() function when if condition is False.

Please let me know if there's a better alternative.

[tssahota]

Hi @vincbeck, I have updated an existing test case to check for a non-null export_arn. Will this suffice with regard to this comment?

[vincbeck]

Could you mock the exported arn using _export_table_to_point_in_time and expect that the returned value is the value you set in the mock?

[vincbeck]

Not sure this is needed

[vincbeck]

Could you mock the exported arn using _export_table_to_point_in_time and expect that the returned value is the value you set in the mock?

[github-actions]

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 5 days if no further activity occurs. Thank you for your contributions.