The GitHub Actions job "Tests (AMD)" on airflow.git/psycopg3-sync-pgvector has 
failed.
Run started by GitHub user Dev-iL (triggered by Dev-iL).

Head commit for run:
4b8a175ba40ef5d445a7eff4b7399584709438e0 / Dev-iL 
<[email protected]>
Guard pgvector operator's psycopg2-specific register_vector import

from pgvector.psycopg2 import register_vector ran unconditionally at
module level, forcing psycopg2 to be importable regardless of which
driver the underlying PostgresHook connection actually uses. Make the
import lazy and raise a clear AirflowOptionalProviderFeatureException
if it's genuinely needed but missing.

This does not fix the separate, pre-existing bug where register_vector
is always called with the psycopg2-specific helper regardless of
whether the connection is actually psycopg2 or psycopg3; that is
tracked at https://github.com/apache/airflow/issues/69443.

The finally block was reimporting the module while the fake psycopg2
absence was still monkeypatched, leaving it cached in a degraded
state for the rest of the test session.

Also documents that this file's two register_vector-mocking tests
depend on psycopg2 staying installed in this provider's dev
environment, which is not guaranteed once apache-airflow-providers-postgres
stops hard-requiring it (a separate, independently-branched fix for the
same migration) -- requesting that provider's [psycopg2] extra here
isn't possible yet since the extra doesn't exist until that fix lands.

Part of the migration tracked in apache/airflow#68453.

Report URL: https://github.com/apache/airflow/actions/runs/28817453608

With regards,
GitHub Actions via GitBox


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to