openzipkin-attic / zipkin-azure

Reporters and collectors for use in Azure
Apache License 2.0
11 stars 9 forks source link

fixing thread-safety issues with single processor handling all partitions #32

Closed aliostad closed 7 years ago

aliostad commented 7 years ago

Hi,

As raised before, current implementation is not thread-safe since data gets received by the processor for all partitions. This small fix allows that one processor gets created for each partition.

It was impossible to add unit test.

Cheers Ali

codefromthecrypt commented 7 years ago

looks good. ack on untestable which is a known issue with the library sadly

appreciated!

aliostad commented 7 years ago

Thanks!