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.

Sign up for GitHub

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

Jump to bottom

Add default limits to queues #6

Closed
alexander-akhmetov opened this issue Dec 10, 2018 · 0 comments
Closed

Add default limits to queues #6

alexander-akhmetov opened this issue Dec 10, 2018 · 0 comments
Assignees
@alexander-akhmetov
Labels
bug Something isn't working enhancement New feature or request

Comments

@alexander-akhmetov
Copy link
Owner

alexander-akhmetov commented Dec 10, 2018 •
edited
Loading

There are no limits for message queues and results:

Message queues
telegram.client.Telegram._queue

python-telegram/telegram/client.py

Line 69 in 80b2619

self._queue: queue.Queue = queue.Queue()

telegram.client.Telegram._workers_queue

python-telegram/telegram/client.py

Line 71 in 80b2619

self._workers_queue: queue.Queue = queue.Queue()

Results dictionary
telegram.client.Telegram._results

python-telegram/telegram/client.py

Line 76 in 80b2619

self._results: Dict[str, AsyncResult] = {}

@alexander-akhmetov alexander-akhmetov self-assigned this Dec 10, 2018
@alexander-akhmetov alexander-akhmetov mentioned this issue Dec 10, 2018
Closed
@alexander-akhmetov alexander-akhmetov added bug Something isn't working enhancement New feature or request labels Dec 10, 2018
alexander-akhmetov pushed a commit that referenced this issue Dec 10, 2018
Merge pull request #9 from alexander-akhmetov/github-6
8ee70d9 
Fix #6 #5 Added default limit for workers queue; Removed unnecessary queue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@alexander-akhmetov alexander-akhmetov

Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@alexander-akhmetov