Skip to content

Commit

Permalink
Clickhouse Connection Argument Nonetype Bug & bump version 1.1.0.3 (#…
Browse files Browse the repository at this point in the history
  • Loading branch information
ulixius9 authored Jul 6, 2023
1 parent 445e094 commit 91189b5
Show file tree
Hide file tree
Showing 5 changed files with 7 additions and 7 deletions.
4 changes: 2 additions & 2 deletions ingestion/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -75,8 +75,8 @@ USER airflow
# Argument to provide for Ingestion Dependencies to install. Defaults to all
ARG INGESTION_DEPENDENCY="all"
RUN pip install --upgrade pip
RUN pip install "openmetadata-managed-apis==1.1.0.2" --constraint "https://raw.githubusercontent.com/apache/airflow/constraints-2.3.3/constraints-3.9.txt"
RUN pip install "openmetadata-ingestion[${INGESTION_DEPENDENCY}]==1.1.0.2"
RUN pip install "openmetadata-managed-apis==1.1.0.3" --constraint "https://raw.githubusercontent.com/apache/airflow/constraints-2.3.3/constraints-3.9.txt"
RUN pip install "openmetadata-ingestion[${INGESTION_DEPENDENCY}]==1.1.0.3"

# Temporary workaround for https://github.com/open-metadata/OpenMetadata/issues/9593
RUN echo "Image built for $(uname -m)"
Expand Down
4 changes: 2 additions & 2 deletions ingestion/operators/docker/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -80,8 +80,8 @@ RUN pip install --upgrade pip

ARG INGESTION_DEPENDENCY="all"
RUN pip install --upgrade pip
RUN pip install "openmetadata-ingestion[airflow]~=1.1.0.2"
RUN pip install "openmetadata-ingestion[${INGESTION_DEPENDENCY}]~=1.1.0.2"
RUN pip install "openmetadata-ingestion[airflow]~=1.1.0.3"
RUN pip install "openmetadata-ingestion[${INGESTION_DEPENDENCY}]~=1.1.0.3"

# Temporary workaround for https://github.com/open-metadata/OpenMetadata/issues/9593
RUN echo "Image built for $(uname -m)"
Expand Down
2 changes: 1 addition & 1 deletion ingestion/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -264,7 +264,7 @@ def get_long_description():
build_options = {"includes": ["_cffi_backend"]}
setup(
name="openmetadata-ingestion",
version="1.1.0.2",
version="1.1.0.3",
url="https://open-metadata.org/",
author="OpenMetadata Committers",
license="Apache License 2.0",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ def get_connection(connection: ClickhouseConnection) -> Engine:
Create Clickhouse connection
"""
if connection.secure or connection.keyfile:
if connection.connectionArguments:
if not connection.connectionArguments:
connection.connectionArguments = init_empty_connection_arguments()
if connection.secure:
connection.connectionArguments.__root__["secure"] = connection.secure
Expand Down
2 changes: 1 addition & 1 deletion openmetadata-airflow-apis/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ def get_long_description():
packages=find_packages(include=[f"{PLUGIN_NAME}.*", PLUGIN_NAME]),
include_package_data=True,
package_data={PLUGIN_NAME: get_package_data()},
version="1.1.0.2",
version="1.1.0.3",
url="https://open-metadata.org/",
author="OpenMetadata Committers",
license="Apache License 2.0",
Expand Down

0 comments on commit 91189b5

Please sign in to comment.