-
-
Notifications
You must be signed in to change notification settings - Fork 5
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
chore: Add bindings for retrieving system metrics #4
base: master
Are you sure you want to change the base?
chore: Add bindings for retrieving system metrics #4
Conversation
Thank you for your contribution.
|
例えこの変更がマージされても |
というのは、可能な限り slacc リポジトリ上にロジックを持ちたくないということを意図しています |
しかし同じような働きをする有名なサードパーティライブラリとしてsysinfoクレートがありますがこれはDisk I/Oの取得に対応していませんしパフォーマンスもこの実装より悪いので代替手段としては検討しづらいのです。私がslaccのコア部分となるクレート・リポジトリを別で作ってslaccをそれに依存させることもできますがそれでよろしいですか? |
リポジトリの構造のみに焦点を当てる限りはそれが望ましいです。
|
わざわざslaccに機能付け加えるのにmisskey-devにリポジトリを作るのはメンテナビリティが下がりそうですしslacc-system-metricsだけではなくslaccの主要機能をCargoのワークスペースで管理したリポジトリを作ることはできませんか? |
と述べたとおりなので、本リポジトリにおいてそれは望ましい状態ではありません |
すみません、これはslacc-system-metricsの機能だけではなくslaccの主要機能をワークスペースで管理したslaccとはまた別のリポジトリのことを指しています。私の理解が正しければこのリポジトリはサードパーティライブラリの薄いラッパーであるslaccを公開するためだけにあるのでワークフローを頻雑化したくないということですよね?こんな感じ:
私はこうしたい:(リポジトリを機能ごとに増やしたくない)
|
Summary
システムメトリクスを取得する高速パスを用意しサーバーメトリクスのパフォーマンスを改善 misskey-dev/misskey#13641
ここで実装していないものはNode.jsのビルトインモジュールを使用するか従来の方法を使用する予定 cc: @syuilo, @acid-chicken