[airflow] branch main updated: Fix PinotDB dependencies (#26705)

[po...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git


The following commit(s) were added to refs/heads/main by this push:
     new 5ff0bc69a5 Fix PinotDB dependencies (#26705)
5ff0bc69a5 is described below

commit 5ff0bc69a5cf09122a91b5e28e72f1a58d443f03
Author: Jarek Potiuk <ja...@potiuk.com>
AuthorDate: Tue Sep 27 13:11:32 2022 +0200

    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
---
 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 219f00d477..be46333706 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 bfbb0a463d..44b5873d22 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": [