From 011cab1d91e66fa250076fae30aa5a37127fd653 Mon Sep 17 00:00:00 2001
From: Jarek Potiuk <jarek@potiuk.com>
Date: Tue, 27 Sep 2022 13:11:32 +0200
Subject: [PATCH] Fix PinotDB dependencies (#26705)

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

GitOrigin-RevId: 5ff0bc69a5cf09122a91b5e28e72f1a58d443f03
---
 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 219f00d4775..be46333706b 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 bfbb0a463dc..44b5873d22a 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": [