search

graphite-project / carbon

Carbon is one of the components of Graphite, and is responsible for receiving metrics over the network and writing them down to disk using a storage backend.
http://graphite.readthedocs.org/
Apache License 2.0
1.5k stars 490 forks source link

remove carbon-client.py, is broken for 4 years #890

Closed piotr1212 closed 4 years ago

piotr1212 commented 4 years ago

Trying to figure out what this does (never used before) looks like it is broken for a long time. Have not seen any complaints so maybe remove it instead of fixing?

Is anybody using this?

deniszh commented 4 years ago

Looks like it's like carbon but takes metrics from stdin. Looks like broken for long time, agreed.

codecov-io commented 4 years ago

Codecov Report

Merging #890 into master will increase coverage by 1.32%. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #890      +/-   ##
==========================================
+ Coverage   50.21%   51.53%   +1.32%     
==========================================
  Files          37       36       -1     
  Lines        3541     3450      -91     
  Branches      508      495      -13     
==========================================
  Hits         1778     1778              
+ Misses       1646     1555      -91     
  Partials      117      117

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 42c1242...1ebd4c1. Read the comment docs.

lgtm-com[bot] commented 4 years ago

This pull request fixes 2 alerts when merging 1ebd4c16cf62bcf6630d77ba593f3f937d278951 into 42c124251e7036bc01ab307c0ac1fb2016a092ab - view on LGTM.com

fixed alerts:

piotr1212 commented 4 years ago

Merging. If someone was actually using this in some old version then let me know.