Skip to content

Commit 839f1e5

Browse files
committed
revert materialized view changes
1 parent b440494 commit 839f1e5

File tree

2 files changed

+5
-7
lines changed

2 files changed

+5
-7
lines changed

functions-python/helpers/database.py

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -152,17 +152,13 @@ def close_db_session(session, raise_exception: bool = False):
152152
raise error
153153

154154

155-
def refresh_materialized_view(session, view_name: str, concurrently=True) -> bool:
155+
def refresh_materialized_view(session, view_name: str) -> bool:
156156
"""
157157
Refresh Materialized view by name.
158158
@return: True if the view was refreshed successfully, False otherwise
159159
"""
160160
try:
161-
session.execute(
162-
text(
163-
f"REFRESH MATERIALIZED VIEW {'CONCURRENTLY' if concurrently else ''} {view_name}"
164-
)
165-
)
161+
session.execute(text(f"REFRESH MATERIALIZED VIEW CONCURRENTLY {view_name}"))
166162
return True
167163
except Exception as error:
168164
logging.error(f"Error raised while refreshing view: {error}")

functions-python/operations_api/src/feeds_operations/impl/feeds_operations_impl.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -151,7 +151,8 @@ async def _update_feed(
151151
else update_request_feed.operational_status_action
152152
)
153153
session.add(feed)
154-
refreshed = refresh_materialized_view(session, t_feedsearch.name, False)
154+
session.flush()
155+
refreshed = refresh_materialized_view(session, t_feedsearch.name)
155156
logging.info(
156157
f"Materialized view {t_feedsearch.name} refreshed: {refreshed}"
157158
)
@@ -170,6 +171,7 @@ async def _update_feed(
170171
logging.error(
171172
f"Failed to update feed ID: {update_request_feed.id}. Error: {e}"
172173
)
174+
session.rollback()
173175
if isinstance(e, HTTPException):
174176
raise e
175177
raise HTTPException(status_code=500, detail=f"Internal server error: {e}")

0 commit comments

Comments
 (0)