Skip to content

[enhance](meta-service)collect fdb kv meta range info as metrics - #52430

Merged
gavinchou merged 12 commits into
apache:masterfrom
koarz:show_meta_ranges
Jul 11, 2025
Merged

[enhance](meta-service)collect fdb kv meta range info as metrics#52430
gavinchou merged 12 commits into
apache:masterfrom
koarz:show_meta_ranges

Conversation

@koarz

@koarzkoarz commented Jun 27, 2025

Copy link
Copy Markdown
Contributor

such as these key:

recycle|default_instance_id|txn: 1
meta|default_instance_id|tablet_index: 1
meta|default_instance_id|rowset_tmp: 1

collect to result:

fdb_kv_ranges_count{category="recycle",instance_id="default_instance_id",sub_category="txn"} 1
fdb_kv_ranges_count{category="meta",instance_id="default_instance_id",sub_category="tablet_index"} 1
fdb_kv_ranges_count{category="meta",instance_id="default_instance_id",sub_category="rowset_tmp"} 1

@hello-stephen

Copy link
Copy Markdown
Contributor

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR.

Please clearly describe your PR:

  1. What problem was fixed (it's best to include specific error reporting information). How it was fixed.
  2. Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
  3. What features were added. Why was this function added?
  4. Which code was refactored and why was this part of the code refactored?
  5. Which functions were optimized and what is the difference before and after the optimization?

@koarz

Copy link
Copy Markdown
ContributorAuthor

run buildall

3 similar comments
@koarz

Copy link
Copy Markdown
ContributorAuthor

run buildall

@koarz

Copy link
Copy Markdown
ContributorAuthor

run buildall

@koarz

Copy link
Copy Markdown
ContributorAuthor

run buildall

@doris-robot

Copy link
Copy Markdown

Cloud UT Coverage Report

Increment line coverage 11.59% (8/69) 🎉

Increment coverage report
Complete coverage report

CategoryCoverage
Function Coverage83.48% (1172/1404)
Line Coverage67.37% (20665/30676)
Region Coverage67.00% (10271/15329)
Branch Coverage56.25% (5374/9554)

@koarz

Copy link
Copy Markdown
ContributorAuthor

run buildall

@hello-stephen

Copy link
Copy Markdown
Contributor

Cloud UT Coverage Report

Increment line coverage 10.67% (8/75) 🎉

Increment coverage report
Complete coverage report

CategoryCoverage
Function Coverage83.48% (1172/1404)
Line Coverage67.41% (20659/30646)
Region Coverage67.01% (10261/15313)
Branch Coverage56.31% (5373/9542)

@koarz

Copy link
Copy Markdown
ContributorAuthor

run cloud_p0

@koarz

Copy link
Copy Markdown
ContributorAuthor

run buildall

@hello-stephen

Copy link
Copy Markdown
Contributor

Cloud UT Coverage Report

Increment line coverage 10.81% (8/74) 🎉

Increment coverage report
Complete coverage report

CategoryCoverage
Function Coverage83.48% (1172/1404)
Line Coverage67.36% (20644/30645)
Region Coverage67.01% (10261/15313)
Branch Coverage56.28% (5370/9542)

Comment threadcloud/src/common/metric.cpp Outdated
Comment threadcloud/src/common/metric.cpp Outdated
Comment threadcloud/src/common/metric.cpp
@koarz

koarz commented Jul 3, 2025

Copy link
Copy Markdown
ContributorAuthor

run buildall

@doris-robot

Copy link
Copy Markdown

Cloud UT Coverage Report

Increment line coverage 8.16% (8/98) 🎉

Increment coverage report
Complete coverage report

CategoryCoverage
Function Coverage82.51% (1189/1441)
Line Coverage67.26% (20747/30847)
Region Coverage66.87% (10317/15429)
Branch Coverage56.19% (5392/9596)

Comment threadcloud/src/meta-service/keys.cpp
Comment threadcloud/src/common/metric.cpp Outdated
@github-actions

Copy link
Copy Markdown
Contributor

PR approved by anyone and no changes requested.

@hello-stephen

Copy link
Copy Markdown
Contributor

Cloud UT Coverage Report

Increment line coverage 10.13% (8/79) 🎉

Increment coverage report
Complete coverage report

CategoryCoverage
Function Coverage81.98% (1269/1548)
Line Coverage67.69% (21616/31936)
Region Coverage67.36% (10732/15932)
Branch Coverage56.87% (5652/9938)

dataroaring
dataroaring previously approved these changes Jul 11, 2025

@dataroaringdataroaring left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@koarz
koarz dismissed stale reviews from dataroaring and gavinchou via f6637feJuly 11, 2025 06:18
@koarz

Copy link
Copy Markdown
ContributorAuthor

run buildall

@github-actionsgithub-actionsBot removed the approved Indicates a PR has been approved by one committer. label Jul 11, 2025
@doris-robot

Copy link
Copy Markdown

Cloud UT Coverage Report

Increment line coverage 10.13% (8/79) 🎉

Increment coverage report
Complete coverage report

CategoryCoverage
Function Coverage82.02% (1273/1552)
Line Coverage67.83% (21749/32062)
Region Coverage67.43% (10762/15960)
Branch Coverage56.97% (5671/9954)

@github-actionsgithub-actionsBot added the approved Indicates a PR has been approved by one committer. label Jul 11, 2025
@github-actions

Copy link
Copy Markdown
Contributor

PR approved by at least one committer and no changes requested.

@gavinchougavinchou changed the title [enhance](meta-service)collect meta range info as metrics[enhance](meta-service)collect fdb kv meta range info as metricsJul 11, 2025
@gavinchou
gavinchou merged commit 5ad2b74 into apache:masterJul 11, 2025
github-actionsBot pushed a commit that referenced this pull request Jul 11, 2025
)
API show_meta_ranges http response
```text
recycle|default_instance_id|txn: 1
meta|default_instance_id|tablet_index: 1
meta|default_instance_id|rowset_tmp: 1
```
metrics
```text
fdb_kv_ranges_count{category="recycle",instance_id="default_instance_id",sub_category="txn"} 1
fdb_kv_ranges_count{category="meta",instance_id="default_instance_id",sub_category="tablet_index"} 1
fdb_kv_ranges_count{category="meta",instance_id="default_instance_id",sub_category="rowset_tmp"} 1
```
@koarz
koarz deleted the show_meta_ranges branch July 11, 2025 08:59
dataroaring pushed a commit that referenced this pull request Jul 12, 2025
…metrics #52430 (#53116)
Cherry-picked from #52430
Co-authored-by: koarz <lihao@selectdb.com>
koarz pushed a commit to koarz/doris that referenced this pull request Jul 14, 2025
…che#52430)
API show_meta_ranges http response
```text
recycle|default_instance_id|txn: 1
meta|default_instance_id|tablet_index: 1
meta|default_instance_id|rowset_tmp: 1
```
metrics
```text
fdb_kv_ranges_count{category="recycle",instance_id="default_instance_id",sub_category="txn"} 1
fdb_kv_ranges_count{category="meta",instance_id="default_instance_id",sub_category="tablet_index"} 1
fdb_kv_ranges_count{category="meta",instance_id="default_instance_id",sub_category="rowset_tmp"} 1
```
morrySnow pushed a commit that referenced this pull request Jul 14, 2025
@gavinchougavinchou mentioned this pull request Aug 15, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approvedIndicates a PR has been approved by one committer.clouddev/3.0.7-mergeddev/3.1.0-mergedreviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@koarz@hello-stephen@doris-robot@gavinchou@Hastyshell@dataroaring@morrySnow