readonly type fix

datalens-tech · Oct 16, 2024 · 8d1e54a · 8d1e54a
1 parent c10e473
commit 8d1e54a
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/lib/dl_connector_clickhouse/dl_connector_clickhouse/core/clickhouse_base/dto.py b/lib/dl_connector_clickhouse/dl_connector_clickhouse/core/clickhouse_base/dto.py
@@ -27,4 +27,4 @@ class ClickHouseConnDTO(ClickHouseBaseDTO, DefaultSQLDTO):  # noqa
     # TODO CONSIDER: Is really optional
     endpoint: Optional[str] = attr.ib(kw_only=True)
     cluster_name: str = attr.ib(kw_only=True)
-    readonly: int = attr.ib(kw_only=True, default=None)
+    readonly: Optional[int] = attr.ib(kw_only=True, default=None)
diff --git a/lib/dl_connector_clickhouse/dl_connector_clickhouse/core/clickhouse_base/target_dto.py b/lib/dl_connector_clickhouse/dl_connector_clickhouse/core/clickhouse_base/target_dto.py
@@ -20,7 +20,7 @@ class BaseClickHouseConnTargetDTO(BaseSQLConnTargetDTO, BaseAiohttpConnTargetDTO
     disable_value_processing: bool = attr.ib()
     secure: bool = attr.ib(kw_only=True, default=False)
     ssl_ca: Optional[str] = attr.ib(kw_only=True, default=None)
-    readonly: int = attr.ib(kw_only=True, default=None)
+    readonly: Optional[int] = attr.ib(kw_only=True, default=None)
 
     # TODO remove in the next release to avoid compatibility issues
     insert_quorum: Optional[int] = attr.ib(kw_only=True, default=None)

diff --git a/lib/dl_connector_clickhouse/dl_connector_clickhouse/core/clickhouse_base/us_connection.py b/lib/dl_connector_clickhouse/dl_connector_clickhouse/core/clickhouse_base/us_connection.py
@@ -54,7 +54,7 @@ class DataModel(ClassicConnectionSQL.DataModel):
         cluster_name: Optional[str] = attr.ib(default=None)
         max_execution_time: Optional[int] = attr.ib(default=None)
         ssl_ca: Optional[str] = attr.ib(kw_only=True, default=None)
-        readonly: int = attr.ib(kw_only=True, default=None)
+        readonly: Optional[int] = attr.ib(kw_only=True, default=None)
 
     def get_conn_dto(self) -> ClickHouseConnDTO:
         return ClickHouseConnDTO(