search

smartcat-labs / cassandra-diagnostics

Cassandra Node Diagnostics Tools
Apache License 2.0
51 stars 6 forks source link

Package for log query report and reporter implementation #73

Closed nbozic-zz closed 8 years ago

nbozic-zz commented 8 years ago

While writing README I noticed that LogQueryReporter is under io.smartcat.cassandra.diagnostics.report.LogQueryReporter and RIemann is under io.smartcat.cassandra.diagnostics.reporter.RiemannQueryReporter. This is confusing, we should have same package name for all reporters.

Alos I noticed that some yaml files have wrong package name for Riemann using report instead of reporter.

nbozic-zz commented 8 years ago

@nivancevic did you see this, how does ft work when it has wrong package name for Riemann reporter?

nbozic-zz commented 8 years ago

And additional question, should LogQueryReporter be part of core or go outside with other implementations