Improve Ad Manager's WaitForReport polling mechanism

Offen

#465 erstellt am 03.6.2020 0 Kommentare 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 3/5
Geschätzter Aufwand: 1-2 Tage
Anfängerfreundlichkeit: 42/100

Issue-Typ: Feature
Klarheit: Größtenteils klar
Aktivitätsstatus: Veraltet
Tech-Stack: python
Bereich: api

Rechercherichtung

Beginne in googleads/ad_manager.py bei AdManagers WaitForReport-Methode, insbesondere beim Aufruf time.sleep(30) und den versionsspezifischen Statusprüfungen. Überprüfe den bestehenden Polling-Ablauf und ermittle das erwartete Backoff-Verhalten; abgeschlossen bedeutet, dass das Report-Polling weiterhin den Status COMPLETED oder FAILED erreicht und dabei zunehmende Verzögerungen verwendet.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

enhancement

Right now AdManager's WaitForReport waits for 30 seconds (via time.sleep(30)) if the report status is not COMPLETED or FAILED before checking the report status again.

Would it be reasonable to implement an exponential backoff strategy instead where we poll for the status of the report with an exponentially increasing delay? Something like the below seems to work:

wait_time_in_seconds = 1
exponential_backoff_multiplier = 2
...

while status != 'COMPLETED' and status != 'FAILED':
     _data_downloader_logger.debug('Report job status: %s', status)

        time.sleep(wait_time_in_seconds)

        if report_downloader._version > 'v201502':
            status = service.getReportJobStatus(report_job_id)
        else:
            status = service.getReportJob(report_job_id)['reportJobStatus']

        # Wait longer if we have to poll the report status again.
        wait_time_in_seconds *= exponential_backoff_multiplier

I will be happy to submit a PR if it makes sense to do the above change.

Vorherrschende Sprache: Python
Sterne: 749
Forks: 967
PR-Merge-Kennzahlen: Keine gemergten PRs in 30 T.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus googleads/googleads-python-lib

[Bug] Potential Billion Laughs Attack Vector via Unrestricted XML Parsing in `ZeepSchemaHelper` Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 45/100

googleads/googleads-python-lib#558 · Erstellt am 12.6.2025
cannot log google ads logging v19 python client Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 35/100

googleads/googleads-python-lib#557 · Erstellt am 13.5.2025
Create Native Creative - Issue Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 25/100

googleads/googleads-python-lib#553 · Erstellt am 28.10.2024
image-package Offen

Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 10/100

googleads/googleads-python-lib#550 · Erstellt am 16.10.2024
Google Cloud Platform /cloud-code-vscode/issues/938 Offen

Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 1/100

googleads/googleads-python-lib#549 · Erstellt am 07.10.2024

Alle Issues in googleads/googleads-python-lib