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

Feature/async knowledge base creation #524

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

sugarforever
Copy link
Owner

Run documents processing in the background.

@satrong
Copy link
Collaborator

satrong commented Jun 16, 2024

开个子进程来处理怎么样?

@sugarforever
Copy link
Owner Author

开个子进程来处理怎么样?

用子进程实现异步处理,能额外带来什么功能/优势?setImmediate是我能想到的最轻量级的实现。

@satrong
Copy link
Collaborator

satrong commented Jun 19, 2024

使用 setImmediate 依然是在主进程里面执行代码吧?

@sugarforever
Copy link
Owner Author

使用 setImmediate 依然是在主进程里面执行代码吧?

是的。这个PR目的只是让前端即时获得反馈。

@satrong
Copy link
Collaborator

satrong commented Jun 19, 2024

占用主进程被会不会堵塞,有待验证

@sugarforever
Copy link
Owner Author

占用主进程被会不会堵塞,有待验证

应该不会,这个我也再验证下。你也可以拉取分支测试一下。

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