Skip to content

Commit

Permalink
Don't create a tcp connection per wsp file.
Browse files Browse the repository at this point in the history
Use one graphite connection per worker instead.
  • Loading branch information
bzed committed Mar 30, 2017
1 parent 5ba4350 commit 400c649
Showing 1 changed file with 12 additions and 7 deletions.
19 changes: 12 additions & 7 deletions dump.go
Original file line number Diff line number Diff line change
Expand Up @@ -43,17 +43,16 @@ func convertFilename(filename string, baseDirectory string) (string, error) {
return "", err
}

func sendWhisperData(filename string, baseDirectory string, graphiteHost string, graphitePort int, graphiteProtocol string) error {
func sendWhisperData(
filename string,
baseDirectory string,
graphiteConn *graphite.Graphite,
) error {
metricName, err := convertFilename(filename, baseDirectory)
if err != nil {
return err
}

graphiteConn, err := graphite.GraphiteFactory(graphiteProtocol, graphiteHost, graphitePort, "")
if err != nil {
return err
}

whisperData, err := whisper.Open(filename)
if err != nil {
return err
Expand Down Expand Up @@ -117,14 +116,20 @@ func worker(ch chan string,

defer wg.Done()

graphiteConn, err := graphite.GraphiteFactory(graphiteProtocol, graphiteHost, graphitePort, "")
if err != nil {
return
}

for {
select {
case path := <-ch:
{

err := sendWhisperData(path, baseDirectory, graphiteHost, graphitePort, graphiteProtocol)
err := sendWhisperData(path, baseDirectory, graphiteConn)
if err != nil {
log.Println("Failed: " + path)
log.Println(err)
} else {
log.Println("OK: " + path)
}
Expand Down

0 comments on commit 400c649

Please sign in to comment.