8000 `from raven import Client` takes 17MB of RAM · Issue #1258 · getsentry/raven-python · GitHub
[go: up one dir, main page]
Skip to content
This repository was archived by the owner on Oct 23, 2023. It is now read-only.

from raven import Client takes 17MB of RAM #1258

Open
skrap opened this issue Jun 7, 2018 · 1 comment
8000
Open

from raven import Client takes 17MB of RAM #1258

skrap opened this issue Jun 7, 2018 · 1 comment
Labels
works-in-new-sdk

Comments

@skrap
Copy link
skrap commented Jun 7, 2018 •
edited
Loading

from raven import Client takes 17MB of RAM

Assumptions:

  • swapless linux system (embedded in my case, but would also affect cloud)
  • python 2.7

To reproduce:

:/tmp# python
Python 2.7.14 (default, Jun  6 2018, 06:11:27) 
[GCC 6.4.0] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import psutil
>>> psutil.Process().memory_info().rss
7974912
>>> from raven import Client
>>> psutil.Process().memory_info().rss
25337856
>>> 25337856-7974912
17362944

The numbers may vary slightly from system to system, but you get the idea.

This comment, from raven.transport.registry, seems relevant:

# TODO(dcramer): we really should need to import all of these by default
from raven.transport.eventlet import EventletHTTPTransport
from raven.transport.exceptions import DuplicateScheme
from raven.transport.http import HTTPTransport
from raven.transport.gevent import GeventedHTTPTransport
from raven.transport.requests import RequestsHTTPTransport
from raven.transport.threaded import ThreadedHTTPTransport
from raven.transport.threaded_requests import ThreadedRequestsHTTPTransport
from raven.transport.twisted import TwistedHTTPTransport
from raven.transport.tornado import TornadoHTTPTransport
from raven.utils import urlparse

(I think dcramer probably meant "should not")
@skrap skrap mentioned this issue Jun 11, 2018
Open
@untitaker
Copy link
Member

Hi @skrap, this should no longer be a problem with the new SDK which only has a single transport implemented: https://github.com/getsentry/sentry-python

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
works-in-new-sdk
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@untitaker @skrap
0