Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP]Add Threading #86

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from
Open

Conversation

shiyutang
Copy link
Collaborator

@shiyutang shiyutang commented Nov 22, 2023

  1. 增加Threading功能,实现消息存储通过user_id 和session_id分离,并可以持久化用户交流信息。
  2. 完成本地测试确认可以进行持久化和隔离化的消息管理。
 graph TD  
  RemoteMemory --> MessageStorageServer  
  MessageStorageServer --> PersistentMessageManager  
  PersistentMessageManager --> Memory  
  MessageManager --> Memory  
  MessageManager --> PersistentMessageManager  
  PersistentMessageManager --> MessageManager
Loading

各模块的定义如下:

  1. RemoteMemory:管理单个session中所有消息;
  2. MessageStorageServer:通过AK确定一个用户中的所有消息历史,并进行session管理;
  3. PersistentMessageManager:持久化消息管理器的本地定义,用户通过这个接口进行持久化、隔离化的消息管理;
  4. MessageManager:不具有持久化管理的消息管理器,和PersistentMessageManager接口一致可以交替使用。

@shiyutang shiyutang changed the title [WIP]Threading [WIP]AddThreading Nov 22, 2023
@shiyutang shiyutang changed the title [WIP]AddThreading [WIP]Add Threading Nov 22, 2023
@shiyutang shiyutang closed this Jan 11, 2024
@shiyutang shiyutang deleted the threading branch January 11, 2024 13:28
@shiyutang shiyutang restored the threading branch January 11, 2024 13:28
@shiyutang shiyutang reopened this Jan 11, 2024
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants