Skip to content

Conversation

@jscheffl
Copy link
Contributor

related: #59402

@boring-cyborg boring-cyborg bot added area:providers provider:edge Edge Executor / Worker (AIP-69) / edge3 labels Dec 14, 2025
@jscheffl jscheffl requested a review from vincbeck December 14, 2025 19:19
@jscheffl jscheffl force-pushed the feature/make-edge-provider-sqla2-compatible branch from 11f9525 to af80318 Compare December 14, 2025 20:03
@jscheffl jscheffl force-pushed the feature/make-edge-provider-sqla2-compatible branch from af80318 to f8060dd Compare December 15, 2025 21:48
@Prab-27
Copy link
Contributor

Prab-27 commented Dec 16, 2025

Nice !!

@jscheffl If possible, could you please verify the mypy errors in step 5? I have updated the issue

@jscheffl jscheffl force-pushed the feature/make-edge-provider-sqla2-compatible branch from f8060dd to 3f321eb Compare December 17, 2025 22:03
@jscheffl
Copy link
Contributor Author

@jscheffl If possible, could you please verify the mypy errors in step 5? I have updated the issue

Uups, yes. overlooked this. Now fixed.

@potiuk potiuk merged commit 50e10d6 into apache:main Dec 17, 2025
82 checks passed
Lohith625 pushed a commit to Lohith625/airflow that referenced this pull request Dec 19, 2025
* Make Edge provider SQLA2 compatible

* Add pre-commit check

* Change filter_by to where

* Uuups, fix

* Review feedback: filter() -> where()

* Review feedback: scalars()....first() -> scalar()

* Fix mypy with SQLA2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:providers provider:edge Edge Executor / Worker (AIP-69) / edge3

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants