fix: Add clickhouse to OFFLINE_STORE_CLASS_FOR_TYPE map (#5251)

* added clickhouse to OFFLINE_STORE_CLASS_FOR_TYPE map

Signed-off-by: Laurynas Stasys <[email protected]>

* removing redundant brackets from from_expression

Signed-off-by: Laurynas Stasys <[email protected]>

---------

Signed-off-by: Laurynas Stasys <[email protected]>
Co-authored-by: Laurynas Stasys <[email protected]>

Author: miroslawas

sdk/python/feast/infra/offline_stores/contrib/clickhouse_offline_store/clickhouse.py

@@ -176,7 +176,7 @@ def pull_latest_from_table_or_query(
             FROM (
                 SELECT {a_field_string},
                 ROW_NUMBER() OVER({partition_by_join_key_string} ORDER BY {timestamp_desc_string}) AS _feast_row
-                FROM ({from_expression}) a
+                FROM {from_expression} a
                 WHERE a."{timestamp_field}"
                     BETWEEN toDateTime64('{start_date.replace(tzinfo=None)!s}', 6, '{start_date.tzinfo!s}')
                     AND toDateTime64('{end_date.replace(tzinfo=None)!s}', 6, '{end_date.tzinfo!s}')

sdk/python/feast/repo_config.py

@@ -100,6 +100,7 @@
     "duckdb": "feast.infra.offline_stores.duckdb.DuckDBOfflineStore",
     "remote": "feast.infra.offline_stores.remote.RemoteOfflineStore",
     "couchbase.offline": "feast.infra.offline_stores.contrib.couchbase_offline_store.couchbase.CouchbaseColumnarOfflineStore",
+    "clickhouse": "feast.infra.offline_stores.contrib.clickhouse_offline_store.clickhouse.ClickhouseOfflineStore",
 }
 
 FEATURE_SERVER_CONFIG_CLASS_FOR_TYPE = {