[
https://issues.apache.org/jira/browse/BEAM-6829?focusedWorklogId=322494&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-322494
]
ASF GitHub Bot logged work on BEAM-6829:
----------------------------------------
Author: ASF GitHub Bot
Created on: 03/Oct/19 09:36
Start Date: 03/Oct/19 09:36
Worklog Time Spent: 10m
Work Description: mxm commented on pull request #8585: [BEAM-6829] Use
transform/pcollection name for metric namespace if none provided
URL: https://github.com/apache/beam/pull/8585#discussion_r330947478
##########
File path:
runners/core-java/src/main/java/org/apache/beam/runners/core/metrics/MonitoringInfoMetricName.java
##########
@@ -51,7 +51,14 @@ private MonitoringInfoMetricName(String urn, Map<String,
String> labels) {
@Override
public String getNamespace() {
if (labels.containsKey(MonitoringInfoConstants.Labels.NAMESPACE)) {
+ // User-generated metric
return labels.getOrDefault(MonitoringInfoConstants.Labels.NAMESPACE,
null);
+ } else if (labels.containsKey(MonitoringInfoConstants.Labels.PCOLLECTION))
{
+ // System-generated metric, prepend with a colon
+ return ":" +
labels.getOrDefault(MonitoringInfoConstants.Labels.PCOLLECTION, null);
+ } else if (labels.containsKey(MonitoringInfoConstants.Labels.PTRANSFORM)) {
+ // System-generated metric, prepend with a colon
+ return ":" +
labels.getOrDefault(MonitoringInfoConstants.Labels.PTRANSFORM, null);
Review comment:
Looking at this again, we may not want to add the colon. My concern here was
possible clash with the user metrics namespace. However, the system metric
names already contain a `beam.` prefix which might be sufficient.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
Issue Time Tracking
-------------------
Worklog Id: (was: 322494)
Time Spent: 2h 50m (was: 2h 40m)
> Duplicate metric warnings clutter log
> -------------------------------------
>
> Key: BEAM-6829
> URL: https://issues.apache.org/jira/browse/BEAM-6829
> Project: Beam
> Issue Type: Bug
> Components: runner-flink
> Affects Versions: 2.11.0
> Reporter: Thomas Weise
> Assignee: Maximilian Michels
> Priority: Major
> Labels: portability
> Time Spent: 2h 50m
> Remaining Estimate: 0h
>
> Logs fill up quickly with these warnings:
> {code:java}
> WARN org.apache.flink.metrics.MetricGroup - Name collision: Group already
> contains a Metric with the name ...{code}
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
