Skip to content

Commit 76e8961

Browse files
committed
fix: short-term-mem postgresql_backend
1 parent bca9483 commit 76e8961

File tree

4 files changed

+8
-50
lines changed

4 files changed

+8
-50
lines changed

pyproject.toml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,8 @@ dependencies = [
3333
"llama-index-llms-openai-like>=0.5.1",
3434
"llama-index-vector-stores-opensearch>=0.6.1",
3535
"llama-index-vector-stores-redis>=0.6.1",
36+
"mcp-server-vikingdb-memory",
37+
"psycopg2-binary>=2.9.10",
3638
]
3739

3840
[project.scripts]
@@ -83,3 +85,6 @@ exclude = [
8385
"veadk/integrations/ve_faas/template/*",
8486
"veadk/integrations/ve_faas/web_template/*"
8587
]
88+
89+
[tool.uv.sources]
90+
mcp-server-vikingdb-memory = { git = "https://github.com/volcengine/mcp-server", subdirectory = "server/mcp_server_vikingdb_memory" }

veadk/memory/short_term_memory.py

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,6 @@
2828
from veadk.memory.short_term_memory_backends.postgresql_backend import (
2929
PostgreSqlSTMBackend,
3030
)
31-
from veadk.memory.short_term_memory_backends.redis_backend import RedisSTMBackend
3231
from veadk.memory.short_term_memory_backends.sqlite_backend import (
3332
SQLiteSTMBackend,
3433
)
@@ -50,10 +49,8 @@ def wrapper(*args, **kwargs):
5049

5150

5251
class ShortTermMemory(BaseModel):
53-
backend: Literal["local", "mysql", "sqlite", "redis", "postgresql", "database"] = (
54-
"local"
55-
)
56-
"""Short term memory backend. `Local` for in-memory storage, `redis` for redis storage, `mysql` for mysql / PostgreSQL storage. `sqlite` for sqlite storage."""
52+
backend: Literal["local", "mysql", "sqlite", "postgresql", "database"] = "local"
53+
"""Short term memory backend. `Local` for in-memory storage, `mysql` for mysql / PostgreSQL storage. `sqlite` for sqlite storage."""
5754

5855
backend_configs: dict = Field(default_factory=dict)
5956
"""Backend specific configurations."""
@@ -90,10 +87,6 @@ def model_post_init(self, __context: Any) -> None:
9087
self._session_service = SQLiteSTMBackend(
9188
local_path=self.local_database_path
9289
).session_service
93-
case "redis":
94-
self._session_service = RedisSTMBackend(
95-
**self.backend_configs
96-
).session_service
9790
case "postgresql":
9891
self._session_service = PostgreSqlSTMBackend(
9992
**self.backend_configs

veadk/memory/short_term_memory_backends/postgresql_backend.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ class PostgreSqlSTMBackend(BaseShortTermMemoryBackend):
3232
postgresql_config: PostgreSqlConfig = Field(default_factory=PostgreSqlConfig)
3333

3434
def model_post_init(self, context: Any) -> None:
35-
self._db_url = f"postgresql+postgresql://{self.postgresql_config.user}:{self.postgresql_config.password}@{self.postgresql_config.host}:{self.postgresql_config.port}/{self.postgresql_config.database}"
35+
self._db_url = f"postgresql+psycopg2://{self.postgresql_config.user}:{self.postgresql_config.password}@{self.postgresql_config.host}:{self.postgresql_config.port}/{self.postgresql_config.database}"
3636

3737
@cached_property
3838
@override

veadk/memory/short_term_memory_backends/redis_backend.py

Lines changed: 0 additions & 40 deletions
This file was deleted.

0 commit comments

Comments
 (0)