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

Add peak and total started thread count to ThreadStatesGaugeSet #1601

Merged
merged 1 commit into from
Jun 11, 2020

Conversation

joschi
Copy link
Member

@joschi joschi commented Jun 11, 2020

Closes #1600

@joschi joschi added this to the 4.1.10 milestone Jun 11, 2020
@joschi joschi requested review from a team June 11, 2020 10:00
@joschi joschi self-assigned this Jun 11, 2020
@jplock jplock merged commit ccc91ef into release/4.1.x Jun 11, 2020
@jplock jplock deleted the issue-1600 branch June 11, 2020 11:06
FMX pushed a commit to apache/celeborn that referenced this pull request Jun 4, 2024
### What changes were proposed in this pull request?

Bump Dropwizard version from 3.2.6 to 4.2.25. Meanwhile, introduce `metrics_jvm_thread_peak_count_Value` and `metrics_jvm_thread_total_started_count_Value` in `celeborn-jvm-dashboard.json`.

### Why are the changes needed?

Dropwizard metrics has released v4.2.25 including some bugfixes and improvements including:

* [JVM] Fix maximum/total memory calculation: dropwizard/metrics#3125
* [Thread] Add peak and total started thread count to `ThreadStatesGaugeSet`: dropwizard/metrics#1601

Meanwhile, Ratis version has upgraded to 3.0.1 which has no compatibility problem with Dropwizard 4.2.25.

Backport:

- apache/spark#26332
- apache/spark#29426
- apache/spark#37372

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

Manual test.

Closes #2540 from SteNicholas/CELEBORN-1389.

Authored-by: SteNicholas <programgeek@163.com>
Signed-off-by: mingji <fengmingxiao.fmx@alibaba-inc.com>
FMX pushed a commit to apache/celeborn that referenced this pull request Jun 4, 2024
### What changes were proposed in this pull request?

Bump Dropwizard version from 3.2.6 to 4.2.25. Meanwhile, introduce `metrics_jvm_thread_peak_count_Value` and `metrics_jvm_thread_total_started_count_Value` in `celeborn-jvm-dashboard.json`.

### Why are the changes needed?

Dropwizard metrics has released v4.2.25 including some bugfixes and improvements including:

* [JVM] Fix maximum/total memory calculation: dropwizard/metrics#3125
* [Thread] Add peak and total started thread count to `ThreadStatesGaugeSet`: dropwizard/metrics#1601

Meanwhile, Ratis version has upgraded to 3.0.1 which has no compatibility problem with Dropwizard 4.2.25.

Backport:

- apache/spark#26332
- apache/spark#29426
- apache/spark#37372

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

Manual test.

Closes #2540 from SteNicholas/CELEBORN-1389.

Authored-by: SteNicholas <programgeek@163.com>
Signed-off-by: mingji <fengmingxiao.fmx@alibaba-inc.com>
(cherry picked from commit 4fc42d7)
Signed-off-by: mingji <fengmingxiao.fmx@alibaba-inc.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

PeakThreadCount information should be added in ThreadStatesGaugeSet
3 participants