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.

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

Clean up errors #96

Closed
tunetheweb opened this issue May 13, 2021 · 0 comments · Fixed by #108
Closed

Clean up errors #96

tunetheweb opened this issue May 13, 2021 · 0 comments · Fixed by #108

Comments

@tunetheweb
Copy link
Member

I hate regular errors in log files. Makes it too easy too miss real errors and difficult for new people to support something as they don't know if they are expected errors or something has gone wrong.

Currently a number of SQL queries cannot run, including:

  • CrUX histograms are based on CrUX data not HTTP Archive crawl so they fail as data usually missing - see CrUX reports not updated since Jan 1st 2020 httparchive.org#306
  • Some of the SQL queries do not work for lenses - for example Blink Usage queries for Capabilities report do not have URL level data to apply lens's.

There's a few things we could do here:

  • Remove reports like CrUX to a separate folder so they can be run separately
  • Fix queries so they do run (e.g. CruUX could be changed to look at previous month's data, Blink Usage reports could add dummy URL column so at least the query doesn't fail, even if it doesn't return data)
  • Add exclusion functionality so certain reports do not run for certain dates or lens's

Any thoughts?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant