keenlabs / capillary

Storm Spout + Kafka State Inspector
MIT License
58 stars 18 forks source link

Fix busted partition iteration in fetching storm spout state. #15

Closed gphat closed 9 years ago

gphat commented 9 years ago

What's This PR Do?

Fixes a bug wherein Capillary wasn't populating all of the partitions when fetching spout state.

Motivation

This was deployed last week and reduced us to getting info for a single partition, oops.

Tests

LOL. Tested against production from my machine. We'll test it in staging first.

terrhorn commented 9 years ago

:+1: