Merge pull request #566 from nurkiewicz/group-name-cadahale-metrics

mattrjacobs · mattrjacobs · commit b48de61ba9fc · 2015-01-25T07:16:10.000-08:00
Hard-coded group name when publishing CodaHale metrics fixed
diff --git a/hystrix-contrib/hystrix-codahale-metrics-publisher/src/main/java/com/netflix/hystrix/contrib/codahalemetricspublisher/HystrixCodaHaleMetricsPublisherCommand.java b/hystrix-contrib/hystrix-codahale-metrics-publisher/src/main/java/com/netflix/hystrix/contrib/codahalemetricspublisher/HystrixCodaHaleMetricsPublisherCommand.java
@@ -43,7 +43,7 @@ public HystrixCodaHaleMetricsPublisherCommand(HystrixCommandKey commandKey, Hyst
         this.circuitBreaker = circuitBreaker;
         this.properties = properties;
         this.metricRegistry = metricRegistry;
-        this.metricGroup = "HystrixCommand";
+        this.metricGroup = commandGroupKey.name();
         this.metricType = key.name();
     }
 

Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,7 @@ public HystrixCodaHaleMetricsPublisherCommand(HystrixCommandKey commandKey, Hyst`
`43`	`43`	`this.circuitBreaker = circuitBreaker;`
`44`	`44`	`this.properties = properties;`
`45`	`45`	`this.metricRegistry = metricRegistry;`
`46`		`- this.metricGroup = "HystrixCommand";`
	`46`	`+ this.metricGroup = commandGroupKey.name();`
`47`	`47`	`this.metricType = key.name();`
`48`	`48`	`}`
`49`	`49`