search

keenlabs / capillary

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

small updates #8

Closed roughley closed 10 years ago

roughley commented 10 years ago

corrected one path and need to check property as a string (not boolean). Tested on trident spouts.

roughley commented 10 years ago

also updated to report metrics on all partitions for a topic, and not just the last one.

gphat commented 10 years ago

Can you explain that last commit @roughley. I don't follow from scanning the code…

roughley commented 10 years ago

There are two thing:

  1. I miss-constructed some of the zookeep paths, and thus causing errors (likely raj1’s issue)
  2. I had > 1 partition, and in this case the tool listed all partitions but only showed data for the last partition.  This was corrected so all partitions display delta, kafka oddest and spout offset

--  Ian Roughley Sent with Airmail

On September 19, 2014 at 12:56:58 PM, Cory G Watson (notifications@github.com) wrote:

Can you explain that last commit @roughley. I don't follow from scanning the code…

— Reply to this email directly or view it on GitHub.