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

perf: add reaction on another thread #66

Merged
merged 1 commit into from
Feb 27, 2024
Merged

perf: add reaction on another thread #66

merged 1 commit into from
Feb 27, 2024

Conversation

yuuahp
Copy link
Member

@yuuahp yuuahp commented Feb 27, 2024

メッセージを送信してから読み上げられるまでが遅いことがあるので調査したところ、リアクション処理に時間がかかっていたので別スレッドでやるようにしました。

大体 0.6~0.8 秒位短縮できるはずです。
(リアクション以外の遅延はほとんど API 呼び出しと Lavaplayer のロードだったので、これでまだ遅かったらうーんです)

@github-actions github-actions bot requested a review from book000 February 27, 2024 12:29
@book000 book000 merged commit 8c674c7 into main Feb 27, 2024
6 checks passed
@book000 book000 deleted the perf/reaction-thread branch February 27, 2024 12:49
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