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

Add BigQuery cookbook #274

Merged
merged 6 commits into from May 17, 2019
Merged

Add BigQuery cookbook #274

merged 6 commits into from May 17, 2019

Conversation

jasonthomas
Copy link
Member

This is my initial pass on BigQuery documentation. Please r? and provide any suggestions and changes.

I think we need more examples but also believe those should probably live in the dataset documentation for the tables.

Copy link

@haroldwoo haroldwoo left a comment

Choose a reason for hiding this comment

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

small nits, otherwise lgtm! TIL about approx funcs

Copy link
Contributor

@mreid-moz mreid-moz left a comment

Choose a reason for hiding this comment

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

I also have a few nits (mostly to appease the spellchecker), otherwise 👍

src/cookbooks/bigquery.md Outdated Show resolved Hide resolved
src/cookbooks/bigquery.md Outdated Show resolved Hide resolved
src/cookbooks/bigquery.md Outdated Show resolved Hide resolved
src/cookbooks/bigquery.md Outdated Show resolved Hide resolved
src/cookbooks/bigquery.md Outdated Show resolved Hide resolved
src/cookbooks/bigquery.md Outdated Show resolved Hide resolved
src/cookbooks/bigquery.md Outdated Show resolved Hide resolved
src/cookbooks/bigquery.md Outdated Show resolved Hide resolved
Copy link
Contributor

@sunahsuh sunahsuh left a comment

Choose a reason for hiding this comment

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

Very exciting, thank you Jason!

Question: do we restrict creating views for these interfaces?

src/cookbooks/bigquery.md Show resolved Hide resolved
src/cookbooks/bigquery.md Outdated Show resolved Hide resolved
src/cookbooks/bigquery.md Show resolved Hide resolved
@jasonthomas
Copy link
Member Author

jasonthomas commented May 15, 2019

Question: do we restrict creating views for these interfaces?

Redash will be completely read-only, but for the rest of the interfaces we should give access to users to create tables and views in the analysis dataset. I will add documentation around that.

@jasonthomas
Copy link
Member Author

I've added the suggestions and requested changes. It would be great if someone can take a quick look before I merge this. Thanks!

@haroldwoo
Copy link

lgtm except the table of contents isn't working?

@jasonthomas
Copy link
Member Author

jasonthomas commented May 17, 2019

It seems to render for me when I run it locally, specifically using mdbook-dtmo serve command. How are you testing?

@haroldwoo
Copy link

haroldwoo commented May 17, 2019 via email

@jasonthomas jasonthomas merged commit 1ec7839 into mozilla:master May 17, 2019
@jasonthomas jasonthomas deleted the bq-1 branch May 17, 2019 16:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants