Skip to content

Commit

Permalink
Fix PinotDB dependencies (#26705)
Browse files Browse the repository at this point in the history
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

python-pinot-dbapi/pinot-dbapi#47

GitOrigin-RevId: 5ff0bc69a5cf09122a91b5e28e72f1a58d443f03
  • Loading branch information
potiuk authored and Cloud Composer Team committed Mar 30, 2023
1 parent 2ffeb1b commit 8fbe140
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 2 deletions.
6 changes: 4 additions & 2 deletions airflow/providers/apache/pinot/provider.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down
1 change: 1 addition & 0 deletions generated/provider_dependencies.json
Original file line number Diff line number Diff line change
Expand Up @@ -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": [
Expand Down

0 comments on commit 8fbe140

Please sign in to comment.