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

Rename "GC Pauses" to "GC Time" in Pulsar JVM dashboards¬ #86

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lhotari
Copy link
Member

@lhotari lhotari commented Jul 22, 2022

@lhotari lhotari requested review from codelipenghui and a team as code owners July 22, 2022 17:59
- source metric is not about GC pauses.¬
- the source metric is "jvm_gc_collection_seconds_sum" which is not the same as "GC pause"¬
- The metric is defined in Prometheus Java client¬
  - "Time spent in a given JVM garbage collector in seconds."¬
  - source code is https://github.com/prometheus/client_java/blob/e68daf23336eb5de7856df406eb1d497f51ad3be/simpleclient_hotspot/src/main/java/io/prometheus/client/hotspot/GarbageCollectorExports.java#L53¬
    - calls GarbageCollectorMXBean.getCollectionTime()¬
      - Javadoc https://docs.oracle.com/en/java/javase/11/docs/api/java.management/java/lang/management/GarbageCollectorMXBean.html#getCollectionTime()¬
        "Returns the approximate accumulated collection elapsed time in milliseconds."¬
     - Stackoverflow Q&A: https://stackoverflow.com/a/44686539¬
@lhotari lhotari force-pushed the lh-rename-gc-pauses-to-collection-time branch from 5cf67d0 to b9488c0 Compare July 22, 2022 20:22
@lhotari lhotari changed the title Rename "GC Pauses" to "GC Collection Time" in Pulsar JVM dashboards¬ Rename "GC Pauses" to "GC Time" in Pulsar JVM dashboards¬ Jul 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant