-
Notifications
You must be signed in to change notification settings - Fork 6
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
ポーリング対象範囲の改善と伴う変数の追加 #133
ポーリング対象範囲の改善と伴う変数の追加 #133
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
パッと見だとこれくらいです
ただUUIDで管理するんじゃなく、前回取得時の最新メッセージの投稿時刻を検索クエリのAfterに入れるっていう案が出ていて、そちらのほうが投稿が削除された場合でも不具合がないのかな、と思っています
最新メッセージの投稿時間を記録しておく方法なら若干ブラックボックス気味な並行処理部分での条件分岐が要らなくなりそうです |
元の実装にかなり近くなった気がします goルーチン中での処理が消えました:iihanashi: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
最新の投稿時刻を保持するにあたって、DBでテーブルを作っておく必要があると思うのでお願いします
現状だと再起動時は最新時刻を起点に収集するはずで、前回の最新投稿の時間を使えないので収集漏れの可能性があります
これ再起動時に何もしてないな(反映させます) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
確認しました
もう少しだけお願いします!:pray:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
あとこれも確認よろしく!
他何か改善点ありましたっけ |
一旦mainブランチの変更点を取り込んでほしいかも |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
問題なさそうです!
改善ありがとうございました!!!
messagePollar周りの改善案です
goルーチンに対する理解が浅いため,並行処理でちゃんと動くか把握できてないので問題があれば教えてください
やってみたこと
メッセージポーリング範囲を
1分追加改善この変更以外が現在の仕様の場合3分ごとに直近4分の投稿を拾う仕様になるmessageProcessorにlastCheckMessage(string),tmplastCheckMessage(string)を追加&それに対応した既存メソッドの変更