Merge branch 'feast-dev:master' into master

feast-dev · Mar 23, 2024 · 6cba191 · 6cba191
2 parents add7f3a + e815562
commit 6cba191
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/sdk/python/feast/infra/offline_stores/contrib/postgres_offline_store/postgres.py b/sdk/python/feast/infra/offline_stores/contrib/postgres_offline_store/postgres.py
@@ -94,7 +94,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 '{start_date}'::timestamptz AND '{end_date}'::timestamptz
             ) b
             WHERE _feast_row = 1

diff --git a/sdk/python/feast/infra/registry/contrib/postgres/postgres_registry_store.py b/sdk/python/feast/infra/registry/contrib/postgres/postgres_registry_store.py
@@ -1,3 +1,4 @@
+import warnings
 from typing import Optional
 
 import psycopg2
@@ -37,6 +38,11 @@ def __init__(self, config: PostgresRegistryConfig, registry_path: str):
             sslcert_path=getattr(config, "sslcert_path", None),
             sslrootcert_path=getattr(config, "sslrootcert_path", None),
         )
+        warnings.warn(
+            "PostgreSQLRegistryStore is deprecated and will be removed in the future releases. Please use SqlRegistry instead.",
+            DeprecationWarning,
+        )
+
         self.table_name = config.path
         self.cache_ttl_seconds = config.cache_ttl_seconds