Skip to content

Commit

Permalink
fix(metrics): incorrect FP rate (#10300)
Browse files Browse the repository at this point in the history
  • Loading branch information
fuyufjh authored Jun 13, 2023
1 parent 54c660b commit 2b2ea49
Show file tree
Hide file tree
Showing 6 changed files with 7 additions and 7 deletions.
2 changes: 1 addition & 1 deletion docker/dashboards/risingwave-dev-dashboard.json

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion docker/dashboards/risingwave-user-dashboard.json

Large diffs are not rendered by default.

4 changes: 2 additions & 2 deletions docs/cpu-profiling.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,13 +6,13 @@ Share an easy-to-use profiler and flamegraph tool: https://github.com/koute/not-
Record samples:

```shell
nperf record -p `pidof compute-node` -o perf.data`
nperf record -p `pidof compute-node` -o perf.data
```

Generate flamegraph:

```shell
nperf flamegraph perf.data > perf.svg
nperf flamegraph --merge-threads perf.data > perf.svg
```

## Profiling remote compute nodes
Expand Down
2 changes: 1 addition & 1 deletion grafana/risingwave-dev-dashboard.dashboard.py
Original file line number Diff line number Diff line change
Expand Up @@ -1792,7 +1792,7 @@ def section_hummock(panels):
"False-Positive / Positive",
[
panels.target(
f"1 - (((sum(rate({table_metric('state_store_read_req_positive_but_non_exist_counts')}[$__rate_interval])) by (job,instance,table_id,type))) / (sum(rate({table_metric('state_store_read_req_bloom_filter_positive_counts')}[$__rate_interval])) by (job,instance,table_id,type)))",
f"(((sum(rate({table_metric('state_store_read_req_positive_but_non_exist_counts')}[$__rate_interval])) by (job,instance,table_id,type))) / (sum(rate({table_metric('state_store_read_req_bloom_filter_positive_counts')}[$__rate_interval])) by (job,instance,table_id,type)))",
"read req bloom filter false positive rate - {{table_id}} - {{type}} @ {{job}} @ {{instance}}",
),
],
Expand Down
2 changes: 1 addition & 1 deletion grafana/risingwave-dev-dashboard.json

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion grafana/risingwave-user-dashboard.json

Large diffs are not rendered by default.

0 comments on commit 2b2ea49

Please sign in to comment.