Skip to content

Fix elasticsearch provider to use SDK imports for Airflow 3.2+#64931

Merged
eladkal merged 1 commit intoapache:mainfrom
potiuk:fix-elasticsearch-sdk-imports
Apr 9, 2026
Merged

Fix elasticsearch provider to use SDK imports for Airflow 3.2+#64931
eladkal merged 1 commit intoapache:mainfrom
potiuk:fix-elasticsearch-sdk-imports

Conversation

@potiuk
Copy link
Copy Markdown
Member

@potiuk potiuk commented Apr 8, 2026

Update es_task_handler.py to import timezone, FileTaskHandler, ExternalLoggingMixin, and LoggingMixin from airflow.sdk.utils when running on Airflow 3.2+, with backwards-compatible fallback to airflow.utils for older versions.


Was generative AI tooling used to co-author this PR?
  • Yes — Claude Opus 4.6

Generated-by: Claude Opus 4.6 following the guidelines

@potiuk potiuk force-pushed the fix-elasticsearch-sdk-imports branch from 5e161fb to 12b98c8 Compare April 8, 2026 22:14
@potiuk potiuk changed the title Fix elasticsearch provider to use SDK imports for Airflow 3.0+ Fix elasticsearch provider to use SDK imports for Airflow 3.2+ Apr 8, 2026
@potiuk potiuk requested a review from amoghrajesh April 8, 2026 22:15
@potiuk potiuk force-pushed the fix-elasticsearch-sdk-imports branch from 12b98c8 to 28eb640 Compare April 9, 2026 00:02
Update es_task_handler.py to import timezone and import_string from
airflow.sdk when running on Airflow 3.2+, with backwards-compatible
fallback to airflow.utils for older versions. Keep FileTaskHandler
and LoggingMixin imports from airflow.utils.log as they remain there.
@potiuk potiuk force-pushed the fix-elasticsearch-sdk-imports branch from 28eb640 to 63fc57d Compare April 9, 2026 00:42
@eladkal eladkal merged commit 6fd0142 into apache:main Apr 9, 2026
103 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants