danielgtaylor / danielgtaylor/python-betterproto

Cannot set timeout in generated `ServiceStubs`

Offen
#245 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Python
Sterne
1.8k
Forks
234
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

It is not possible to set `timeout`s in generated `ServiceStubs`.

I believe betterproto supports timeouts since #32 but it seems like the code generation does not implement timeouts.

[Here](https://github.com/qria/python-betterproto/tree/feature/add-timeout) is my fork that I am currently using to use timeouts.

I didn't submit a PR because it seems like there's much context, such as 'does timeout only apply in case of `not method.client_streaming and method.py_input_message and method.py_input_message.fields`?

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Vergleiche die generierten ServiceStubs mit der in Issue #32 referenzierten Timeout-Unterstützung und dem verknüpften feature/add-timeout-Fork. Stelle zunächst fest, welche generierten Methoden Timeouts akzeptieren oder anwenden sollten und ob Streaming- oder Bedingungen zur Nachrichtenform diesen Umfang beeinflussen; abgeschlossen ist die Aufgabe, wenn das erwartete Timeout-Verhalten definiert und in den generierten Stubs konsistent implementiert ist.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
grpc, python
Bereich
api, tooling
Issue-Typ
Feature
Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.