it fixes metrics overflow, causing time bug

Author: GriwMF

README.md

@@ -105,6 +105,8 @@ There are four different types of metrics:
 
         t = Time.now
         Stackify::Metrics.time "MyCategory", "ElapsedTime", t
+        # or
+        Stackify::Metrics.time_duration "MyCategory", "ElapsedTime", 5.seconds
 
 - **Counter and Timer**: Composite of the Counter and Timer metrics for convenience
 

lib/stackify/metrics/metrics_client.rb

@@ -61,7 +61,7 @@ def start_upload_metrics
       read_queued_metrics_batch current_time
       handle_zero_reports current_time
 
-      selected_aggr_metrics = @aggregate_metrics.select { |_key, aggr| aggr.occurred_utc < current_time }
+      selected_aggr_metrics = @aggregate_metrics.select { |_key, aggr| aggr.occurred_utc.to_i < current_time.to_i && !aggr.sent }
       first_50_metrics = Hash[selected_aggr_metrics.to_a.take 50]
       if first_50_metrics.length > 0
         #only getting metrics less than 10 minutes old to drop old data in case we get backed up
@@ -111,7 +111,7 @@ def handle_zero_reports current_time
             @metric_settings.delete[aggregate.name_key]
             next
           end
-          agg = aggregate.dup
+          agg = aggregate.clone
           agg.occurred_utc = current_time
 
           disabled_autoreport_last = [
@@ -182,12 +182,12 @@ def upload_aggregates aggr_metrics
         else
           metric.monitor_id = mon_info['MonitorID']
         end
-
-        #get identified once
-        aggr_metrics_for_upload = aggr_metrics.select { |_key, aggr_metric| !aggr_metric.monitor_id.nil? }
-        @metrics_sender.upload_metrics aggr_metrics_for_upload
-        all_success
       end
+
+      #get identified once
+      aggr_metrics_for_upload = aggr_metrics.select { |_key, aggr_metric| !aggr_metric.monitor_id.nil? }
+      @metrics_sender.upload_metrics aggr_metrics_for_upload
+      all_success
     end
   end
 end