openzipkin / zipkin-aws

Reporters and collectors for use in Amazon's cloud
Apache License 2.0
69 stars 34 forks source link

SQS Collector test flake #82

Open devinsba opened 6 years ago

devinsba commented 6 years ago

When building my PR, that doesn't touch SQS, I encountered a test flake in the new tests added in #79

Not sure what happened. The circle build can be seen here: https://circleci.com/gh/openzipkin/zipkin-aws/426

cc @adriancole @llinder

llinder commented 6 years ago

Thanks for raising the issue. I think the test I added for deserialization errors is a bit unreliable on the CI server. I will look at fixing this up.