Skip to content

Commit

Permalink
Remove Atomic variable as it is not required.
Browse files Browse the repository at this point in the history
  • Loading branch information
wolfeidau committed Oct 26, 2016
1 parent 2e8253c commit 6952acb
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions plugins/outputs/kinesis/kinesis.go
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ package kinesis
import (
"log"
"os"
"sync/atomic"
"time"

"github.com/aws/aws-sdk-go/aws"
Expand Down Expand Up @@ -158,7 +157,7 @@ func writekinesis(k *KinesisOutput, r []*kinesis.PutRecordsRequestEntry) time.Du
}

func (k *KinesisOutput) Write(metrics []telegraf.Metric) error {
var sz uint32 = 0
var sz uint32

if len(metrics) == 0 {
return nil
Expand All @@ -167,7 +166,7 @@ func (k *KinesisOutput) Write(metrics []telegraf.Metric) error {
r := []*kinesis.PutRecordsRequestEntry{}

for _, metric := range metrics {
atomic.AddUint32(&sz, 1)
sz++

values, err := k.serializer.Serialize(metric)
if err != nil {
Expand All @@ -185,7 +184,7 @@ func (k *KinesisOutput) Write(metrics []telegraf.Metric) error {
// Max Messages Per PutRecordRequest is 500
elapsed := writekinesis(k, r)
log.Printf("E! Wrote a %+v point batch to Kinesis in %+v.\n", sz, elapsed)
atomic.StoreUint32(&sz, 0)
sz = 0
r = nil
}
}
Expand Down

0 comments on commit 6952acb

Please sign in to comment.