Skip to content

fix(bug-2060144): resolve google play store bad request error - #9762

Merged
kik-kik merged 5 commits into
mainfrom
fix/bug-2060144/resolve-google_play_store-bad-request
Aug 6, 2026
Merged

fix(bug-2060144): resolve google play store bad request error#9762
kik-kik merged 5 commits into
mainfrom
fix/bug-2060144/resolve-google_play_store-bad-request

Conversation

@kik-kik

@kik-kik kik-kik commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

fix(bug-2060144): resolve google play store bad request error

Description

This makes the following three changes:

  1. Improve logging of the HTTP error response to make it easier to understand why a request failed.
  2. Use a 7 day lag for this job, it appears there is a latency on Google side in making this data available (as of writing the earliest date available is 30th of July).
  3. Change the tier tag from tier_2 to tier_3.

@kik-kik kik-kik self-assigned this Aug 5, 2026
@kik-kik kik-kik added the bug Something isn't working label Aug 5, 2026

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR wraps response.raise_for_status() in the six Google Play Store query.py jobs to log the response body before re-raising, moves all seven google_play_store_derived tables to a 7-day scheduling lag, and retags the bqetl_google_play_store DAG from impact/tier_2 to impact/tier_3.

The lag change is consistent for the four vitals_* tables, but the three slow_startup_* scripts apply an additional -1 day shift internally, so the partition they write no longer matches the date_partition_offset being declared — details inline. The error-handling block also calls response.json() on a response that may not be JSON, inside the exception handler.

Comment thread sql/moz-fx-data-shared-prod/google_play_store_derived/vitals_anr_rate_v1/query.py Outdated
@scholtzan

This comment has been minimized.

@scholtzan

This comment has been minimized.

@lucia-vargas-a lucia-vargas-a left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • comment on removing the timedelta

@kik-kik
kik-kik force-pushed the fix/bug-2060144/resolve-google_play_store-bad-request branch from 447c671 to da0f91c Compare August 6, 2026 08:11
@scholtzan

This comment has been minimized.

@kik-kik
kik-kik force-pushed the fix/bug-2060144/resolve-google_play_store-bad-request branch from da0f91c to 275df7e Compare August 6, 2026 13:44
@kik-kik
kik-kik added this pull request to the merge queue Aug 6, 2026
@scholtzan

Copy link
Copy Markdown
Collaborator

Integration report

Merged via the queue into main with commit 200e641 Aug 6, 2026
30 checks passed
@kik-kik
kik-kik deleted the fix/bug-2060144/resolve-google_play_store-bad-request branch August 6, 2026 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants