creating a new client at every status check

[pedrovsg]

Seems we are always creating a new client every time.
I think we should do it only once.

go-pubsub/kafka/status.go

Line 11 in 8ce052d

client, err := sarama.NewClient(brokerAddrs, sarama.NewConfig())

[aneshas]

Would you guys accept a PR for this if it does turn out to introduce unnecessary overhead :) ?

[mjgarton]

We'd consider it of course.

I should let you know though that go-pubsub developement is slowing somewhat now, due to https://github.com/uw-labs/substrate which is intented to eventually replace go-pubsub and address some of it's shortcomings, although substrate still has a few things missing at this point.