Fix map duplicates (#55)

* ensure no duplicates in map

* forgot imports
chose better variable name

* replace putall logic with foreach on stream

Author: PhoenixRion

src/main/java/com/arpnetworking/clusteraggregator/client/AggClientConnection.java

@@ -193,7 +193,16 @@ private Optional<PeriodicData> buildPeriodicData(final Messages.StatisticSetReco
         final CombinedMetricData combinedMetricData = CombinedMetricData.Builder.fromStatisticSetRecord(setRecord).build();
         final ImmutableList.Builder<AggregatedData> builder = ImmutableList.builder();
         final Map<String, String> dimensionsMap = setRecord.getDimensionsMap();
-        final ImmutableMap.Builder<String, String> dimensionBuilder = ImmutableMap.<String, String>builder().putAll(dimensionsMap);
+        final ImmutableMap.Builder<String, String> dimensionBuilder = ImmutableMap.builder();
+
+        dimensionsMap.entrySet().stream()
+                .filter(entry ->
+                        !CombinedMetricData.HOST_KEY.equals(entry.getKey())
+                        && !CombinedMetricData.SERVICE_KEY.equals(entry.getKey())
+                        && !CombinedMetricData.CLUSTER_KEY.equals(entry.getKey()))
+                .forEach(dim ->
+                        dimensionBuilder.put(dim.getKey(), dim.getValue()
+                ));
 
         Optional<String> host = Optional.ofNullable(dimensionsMap.get(CombinedMetricData.HOST_KEY));
         Optional<String> service = Optional.ofNullable(dimensionsMap.get(CombinedMetricData.SERVICE_KEY));