From da389744b355255b2c3737c41a0682b8931c1c22 Mon Sep 17 00:00:00 2001 From: Jarek Potiuk Date: Tue, 27 Sep 2022 12:16:47 +0200 Subject: [PATCH] Fix PinotDB dependencies Our canary builds detected that PinotDB released 7 hours ago lacks ciso8601 dependency, which make import fail during test collection. This PR adds the dependency in provider until the issue is fixed. PR has been raised to PinotDB https://github.com/python-pinot-dbapi/pinot-dbapi/issues/47 --- airflow/providers/apache/pinot/provider.yaml | 6 ++++-- generated/provider_dependencies.json | 1 + 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/airflow/providers/apache/pinot/provider.yaml b/airflow/providers/apache/pinot/provider.yaml index 219f00d477588..be46333706b3d 100644 --- a/airflow/providers/apache/pinot/provider.yaml +++ b/airflow/providers/apache/pinot/provider.yaml @@ -36,9 +36,11 @@ versions: dependencies: - apache-airflow>=2.2.0 - apache-airflow-providers-common-sql>=1.1.0 - # pinotdb v0.1.1 may still work with older versions of Apache Pinot, but we've confirmed that it - # causes a problem with newer versions. - pinotdb>0.1.2 + # Pinotdb 0.4.7 lacks ciso8601 as install_requires dependency, yet it has implicit dependency on it + # Until https://github.com/python-pinot-dbapi/pinot-dbapi/issues/47 is fixed we add the dependency in + # the provider dependency + - ciso8601 integrations: - integration-name: Apache Pinot diff --git a/generated/provider_dependencies.json b/generated/provider_dependencies.json index bfbb0a463dcd1..44b5873d22a51 100644 --- a/generated/provider_dependencies.json +++ b/generated/provider_dependencies.json @@ -133,6 +133,7 @@ "deps": [ "apache-airflow-providers-common-sql>=1.1.0", "apache-airflow>=2.2.0", + "ciso8601", "pinotdb>0.1.2" ], "cross-providers-deps": [