Pure Python `SimpleQueue.get()` raises `ValueError` when block is false and timeout is given

Ouverte
#156,476 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
25/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
À l'abandon
Stack technique
python
Domaine
backend

Piste de recherche

Commencez par l’implémentation en Python pur de _PySimpleQueue.get() et comparez la gestion de block=False et timeout avec la version C et la documentation de SimpleQueue. C’est terminé lorsqu’un get non bloquant avec un timeout fourni suit le comportement documenté et lève Empty plutôt que ValueError ; vérifiez le comportement existant présenté dans l’issue.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

stdlib type-bug

The docs for SimpleQueue.get() say that timeout is ignored when block is false. The C version does that, but the pure Python version passes timeout to threading.Semaphore.acquire() and fails:

>>> from queue import _PySimpleQueue
>>> _PySimpleQueue().get(block=False, timeout=1)
Traceback (most recent call last):
  ...
ValueError: can't specify timeout for non-blocking acquire

The C version raises Empty here.

A PR is on the way.

Linked PRs
  • gh-156477
  • gh-156487
  • gh-156488
  • gh-156489
Langage dominant
Python
Étoiles
77.2k
Forks
36k
Merge moyen
1 j 9 h
PR mergées (30 j)
558

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de python/cpython

Toutes les issues de python/cpython

Issues similaires

Plus d'issues Python

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.