Make configure_content functional. Do not pass content index state to it.

This commit is contained in:
Debanjum Singh Solanky
2023-07-20 23:13:16 -07:00
parent e92bc0e2e6
commit 2ddee7e745

View File

@@ -85,7 +85,7 @@ def configure_server(config: FullConfig, regenerate: bool, search_type: Optional
try:
state.config_lock.acquire()
state.content_index = configure_content(
state.content_index, state.config.content_type, state.search_models, regenerate, search_type
state.config.content_type, state.search_models, regenerate, search_type
)
except Exception as e:
logger.error(f"🚨 Failed to index content")
@@ -112,9 +112,7 @@ if not state.demo:
def update_search_index():
try:
state.config_lock.acquire()
state.content_index = configure_content(
state.content_index, state.config.content_type, state.search_models, regenerate=False
)
state.content_index = configure_content(state.config.content_type, state.search_models, regenerate=False)
logger.info("📬 Content index updated via Scheduler")
except Exception as e:
logger.error(f"🚨 Error updating content index via Scheduler: {e}")
@@ -155,7 +153,6 @@ def configure_search(search_models: SearchModels, search_config: Optional[Search
def configure_content(
content_index: Optional[ContentIndex],
content_config: Optional[ContentConfig],
search_models: SearchModels,
regenerate: bool,
@@ -165,8 +162,9 @@ def configure_content(
if content_config is None:
logger.warning("🚨 No Content configuration available.")
return None
if content_index is None:
content_index = ContentIndex()
# Initialize Variables
content_index = ContentIndex()
try:
# Initialize Org Notes Search