From b4a1d14a12fe301fce3ad9279a9190968aaf9c29 Mon Sep 17 00:00:00 2001 From: leenr Date: Sat, 4 Mar 2017 15:43:38 +0300 Subject: [PATCH] Make ver_col in ReplacingMergeTree optional --- src/infi/clickhouse_orm/engines.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/infi/clickhouse_orm/engines.py b/src/infi/clickhouse_orm/engines.py index 9157752..012152f 100644 --- a/src/infi/clickhouse_orm/engines.py +++ b/src/infi/clickhouse_orm/engines.py @@ -65,14 +65,15 @@ class SummingMergeTree(MergeTree): class ReplacingMergeTree(MergeTree): - def __init__(self, date_col, key_cols, ver_col, sampling_expr=None, + def __init__(self, date_col, key_cols, ver_col=None, sampling_expr=None, index_granularity=8192, replica_table_path=None, replica_name=None): super(ReplacingMergeTree, self).__init__(date_col, key_cols, sampling_expr, index_granularity, replica_table_path, replica_name) self.ver_col = ver_col def _build_sql_params(self): params = super(ReplacingMergeTree, self)._build_sql_params() - params.append(self.ver_col) + if self.ver_col: + params.append(self.ver_col) return params