Fixes on getting started BQ article - #23
Conversation
- Removed two sections that were not needed and/or were incomplete/incorrect. - Removed two links that were not needed - Updated the query to use median instead of average, and query only one day - Fixed the numbers for size of BQ per day
|
Looks good to me. |
jduckles
left a comment
jduckles
left a comment
There was a problem hiding this comment.
I'm fine with most of these change, just curious why we're removing links to outside documentation.
| ## Further Reading | ||
|
|
||
| - [M-Lab BigQuery Schema](https://www.measurementlab.net/data/docs/bq/schema) — full schema documentation | ||
| - [Google BigQuery documentation](https://cloud.google.com/bigquery/what-is-bigquery) |
There was a problem hiding this comment.
Not sure why we're removing links to content that can help our user's and community members place jargon (like BigQuery) in a broader context.
There was a problem hiding this comment.
It's the level of detail / tech depth of the article. We assume users have basic understading of these systems. Otherwise half of the content cannot be understood. Or, alternatively, we would need to provide links for all systems referred in there (e.g., Google Cloud Console, SKD, etc.)
However, you're correct that we should help our users, in case they are not familiar with these systems. I'm adding this link in the beginning of the article, inline with the text, similarly to the Google Cloud Console, etc.
2 participants
Removed two sections that were not needed and/or were incomplete/incorrect.
Removed two links that were not needed
Updated the query to use median instead of average, and query only one day
Fixed the numbers for size of BQ per day