-
Notifications
You must be signed in to change notification settings - Fork 28.2k
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
[SPARK-12983] [CORE] [DOC] Correct metrics.properties.template #10902
Conversation
# "/metrics/applications/json" can be sent seperately to get metrics snapshot of | ||
# instance master and applications. MetricsServlet may not be configured by self. | ||
# | ||
# 5. The MetricsServlet sink is added by default as a sink in master, worker and |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
While we're here...
- in the master
- , and you can send
- json -> JSON
LGTM, the rest are neutral or small improvements |
Test build #49999 has finished for PR 10902 at commit
|
Test build #50018 has finished for PR 10902 at commit
|
Unrelated to this pr, triggering a new build. |
Jenkins, retest this please. |
Test build #50075 has finished for PR 10902 at commit
|
cc @srowen and @JoshRosen since this is the spinoff of #10787` |
merged to master |
There are some typos or plain unintelligible sentences in the metrics template.