Skip to content

Fix incorrect documented defaults in Amazon operators and sensors - #72315

Merged
Lee-W merged 1 commit into
apache:mainfrom
FrankYang0529:airflow-amazon-docstring-defaults
Sep 7, 2026
Merged

Lee-W merged 1 commit into
apache:mainfrom
FrankYang0529:airflow-amazon-docstring-defaults

Conversation

@FrankYang0529

Copy link
Copy Markdown
Member

The class docstring is different from class default value.

Class Parameter Documented Actual
GlueJobOperator waiter_max_attempts 20 75
BedrockBatchInferenceOperator waiter_max_attempts 10 20
BedrockBatchInferenceSensor poke_interval 5 120
BedrockBatchInferenceSensor max_retries 24 75
SqsPublishOperator delay_seconds 1 second 0

Was generative AI tooling used to co-author this PR?
  • Yes - Claude Code

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.

Signed-off-by: PoAn Yang <payang@apache.org>
@boring-cyborg boring-cyborg Bot added area:providers provider:amazon AWS/Amazon - related issues labels Aug 31, 2026
@FrankYang0529
FrankYang0529 marked this pull request as ready for review August 31, 2026 07:53
@Lee-W
Lee-W merged commit 80797fa into apache:main Sep 7, 2026
160 checks passed
@FrankYang0529
FrankYang0529 deleted the airflow-amazon-docstring-defaults branch September 7, 2026 07:25
pierrejeambrun pushed a commit to astronomer/airflow that referenced this pull request Sep 8, 2026
imrichardwu pushed a commit to imrichardwu/airflow that referenced this pull request Sep 11, 2026
regarmukesh3g pushed a commit to regarmukesh3g/airflow that referenced this pull request Sep 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:providers provider:amazon AWS/Amazon - related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants