twilio / twilio/twilio-ruby

twilio-ruby is taking too much memory.

Open
#396 30 comments 28 reactions 1 assignee View on GitHub

@philnash is already working on this.

Since Oct 22, 2018.

difficulty: hard status: work in progress type: twilio enhancement
Dominant language
Ruby
Stars
1.4k
Forks
472
Avg merge
1d 3h
Merged PRs (30d)
5

Description

Version: 5.8

twilio-ruby is taking too much memory.

Steps to Reproduce

I audited my project dependencies with the help of derailed_benchmarks and I was surprised too find twilio-ruby at the top of mem usage even before the rails stack.

Here is an extract of the derailed gem

$  bundle exec derailed bundle:mem

TOP: 89.0391 MiB
  twilio-ruby: 28.1602 MiB
    ~/.rvm/gems/ruby-2.3.1/gems/twilio-ruby-5.8.0/lib/twilio-ruby/rest/ip_messaging/v1/service.rb: 0.3047 MiB
    ~/.rvm/gems/ruby-2.3.1/gems/twilio-ruby-5.8.0/lib/twilio-ruby/rest/api/v2010/account.rb: 0.3008 MiB
  rails/all: 20.332 MiB
    rails: 9.3125 MiB (Also required by: active_record/railtie, active_model/railtie, and 8 others)
...

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.