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

ジョブキューのチャートを更新するときに現在のジョブ数を取得するのではなく前回の更新時からのイベントを受信しておきメモリにカウントを保持しておいてそれを利用する #4470

Closed
syuilo opened this issue Mar 11, 2019 · 2 comments
Labels
✨Feature This adds/improves/enhances a feature packages/backend Server side specific issue/PR

Comments

@syuilo
Copy link
Member

syuilo commented Mar 11, 2019

Summary

より正確な数を取得するため

@syuilo syuilo added ✨Feature This adds/improves/enhances a feature packages/backend Server side specific issue/PR labels Mar 11, 2019
@syuilo syuilo closed this as completed in e7f81a4 Mar 12, 2019
@mei23
Copy link
Contributor

mei23 commented Mar 12, 2019

「現在Activeなジョブ数 (count)」と「単位時間あたりにActiveになったジョブ数 (count/T sec)」
は違う概念なので代替えにはならないのではと思います。
「単位時間あたりのActiveになったジョブ数」では来客数的なのはわかりますが
滞留しているかがわからないので「現在Activeなジョブ数」も必要だと思います。

また「現在Activeなジョブ数 (count)」と「単位時間あたりにActiveになったジョブ数 (count/T sec)」は単位の次元?も違うので、同じグラフに入れてしまうのも少し変な気がします。

つまり「現在Activeなジョブ数 (count)」は残して
「単位時間あたりにActiveになったジョブ数 (count/T sec)」は別枠の集計としたほうがいいのではと思います。

@mei23
Copy link
Contributor

mei23 commented Mar 12, 2019

おそらくここで欲しいと思っている値は「Activeなジョブ数の単位時間あたりの平均値 (count) 」で
それは
「単位時間あたりにActiveになった数 (count/T)」x「単位時間あたりに処理されたジョブの所要時間の平均(T)」で求まるのではと思われます(たしか)。

でもこれ取得できなそう…

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨Feature This adds/improves/enhances a feature packages/backend Server side specific issue/PR
Projects
None yet
Development

No branches or pull requests

2 participants