zipkin-sink lost span

[leonly0224]

the sink lost span while sending them to zipkin collecter

cased by class ZipkinSpanCollectorSink on line 114
while ((event = channel.take()) != null && count < batchSize) {

some events are already taken from the channel even count < batchSize is false
should be changed to
while (count < batchSize && (event = channel.take()) != null) {

[kristofa]

Good catch! I'll try to fix this tonight!

[kristofa]

Bugfix is checked in. Problem is that there are no unit tests for flume sinks because they are a pain to test because of the inheriting from AbstractSink. But I might add tests later anyway to ensure these kinds of bugs do not recur.

Thanks again!