Fixed the issue #9, SDI-1522

intelsdi-x / snap-plugin-collector-elasticsearch

Collects Elasticsearch cluster and nodes statistics

http://snap-telemetry.io/

Apache License 2.0

3 stars 14 forks source link

Closed candysmurf closed 8 years ago

candysmurf commented 8 years ago

Summary:

Test done:

nanliu commented 8 years ago

Changes looks ok, we may need to review why namespace.String decided to add a leading slash and whether this affects other plugins: https://github.com/intelsdi-x/snap/blob/master/core/metric.go#L50-L55

candysmurf commented 8 years ago

@nanliu, thanks Nan for reviewing it. Your observation is very good. I'll follow up with Iza to see any impact to other plugins.