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

添加 get_message 动作 #167

Closed
wants to merge 2 commits into from
Closed

Conversation

abrahum
Copy link
Collaborator

@abrahum abrahum commented Apr 29, 2022

@crazywhalecc
Copy link
Member

此接口可能需要规范一下历史数据和获取无效数据的问题,如果只写了接口的定义可能会有部分歧义,例如该消息不存在时应该返回什么内容。此外,考虑到不同语言上对缓存 DB 的支持不同,有些平台的 libob 可能没有使用 DB,只使用了如内存变量等方式进行缓存,而即使是数据库也可能不会储存所有历史的消息,或者没有构建所需的 message_id => MessageEvent 对应的 Map 或 Table,频繁调用此接口可能会很影响性能,虽然此处应该是实现本身的问题,但对于 libob 的开发也是一个挑战。

@stdrc
Copy link
Member

stdrc commented Jun 13, 2022

虽然此处应该是实现本身的问题,但对于 libob 的开发也是一个挑战。

如果这个接口进标准,感觉 libob 就需要加上内置数据库了

@crazywhalecc
Copy link
Member

如果这个接口进标准,感觉 libob 就需要加上内置数据库了

主要是这个接口其实本身和 OneBot 实现关系不大,应该交由 libob 去做。

@abrahum abrahum closed this Jul 1, 2022
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.

3 participants