tikv/tikv

Better memory observation for tidb_query executors

Open

#5,338 opened on 2019年8月27日

GitHub で見る
 (7 comments) (0 reactions) (0 assignees)Rust (15,922 stars) (2,189 forks)batch import
difficulty/easyhelp wantedsig/coprocessorstatus/mentorstatus/proposal

説明

Feature Request

Inspired from https://github.com/pingcap/tidb/issues/11840. Currently we have no existing way to check how much memory is used in some memory-heavy executors like hash aggregation and top n. Adding metrics to these executors should help diagnose these kind of problems without the need to build a specialized binary.

コントリビューターガイド

技術スタック
rust
領域
backendperformanceobservability
Issue 種別
feature
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
4
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
3-5 days
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
stale
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
mostly clear
前提条件
Rust programmingKnowledge of TiKV codebaseUnderstanding of memory profiling
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
25
調査方針
Study the existing memory metrics in TiKV, particularly in the tidb query executor components. Look at the hash aggregation and top n executors in the tikv/src/coprocessor directory. Examine how metrics are implemented elsewhere, for example in tikv/src/server/status.rs using the prometheus crate. Identify where to add memory monitoring and propose specific metrics (e.g., cumulative memory usage, peak memory).