You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Perhaps add a protocol='pickle' option to Sender.__init__ (defaults to plaintext). It'd be nice if the port argument defaulted to None, and then if None we'd select the default port based on protocol (plaintext=2003, pickle=2004).
If it just sent one at a time in foreground (interval=None) mode, I think that's fine, and the benefits would only kick in if you were in background (interval) mode. But I think that's fine.
The text was updated successfully, but these errors were encountered:
The pickle protocol is somewhat more efficient, especially when sending a bunch of metrics at once. Would be nice to add an option to send data using the pickle format, as per http://graphite.readthedocs.io/en/latest/feeding-carbon.html:
Perhaps add a
protocol='pickle'
option toSender.__init__
(defaults toplaintext
). It'd be nice if theport
argument defaulted to None, and then if None we'd select the default port based on protocol (plaintext=2003, pickle=2004).If it just sent one at a time in foreground (interval=None) mode, I think that's fine, and the benefits would only kick in if you were in background (interval) mode. But I think that's fine.
The text was updated successfully, but these errors were encountered: