Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open to option to enable telemetry by option? #42

Open
travisgriggs opened this issue Dec 12, 2022 · 1 comment
Open

Open to option to enable telemetry by option? #42

travisgriggs opened this issue Dec 12, 2022 · 1 comment

Comments

@travisgriggs
Copy link

Every tortoise/mqtt connection spins up a telemetry process. There's not much to it. It periodically (30seconds I think) samples the socket and updates total sent/receive bytes. I have no use for this. But I spin up lots of connections, so all of those little processes add up as far as memory consumption.

Right now, I'm just commenting out the child_spec entry in my local deps directory and things work fine. Would there be openness to making this an option of the connection (defaulted to true of course so behavior stays same)? If so, I'll do a pull-request, otherwise, I'll just keep hacking it locally.

@jfcloutier
Copy link
Collaborator

Please go ahead with the PR. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants