-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
doc(redshift) - Adding Redshift ingestion quickstart guide #7700
doc(redshift) - Adding Redshift ingestion quickstart guide #7700
Conversation
Small fix for BigQuery and Snowflake doc
|
||
## Configure Recipe | ||
|
||
5. Navigate to the **Sources** tab and click **Create new source** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should not be step 5, right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be a new set of steps, or is that we are doing?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
in other docs
* [BigQuery Resource Viewer](https://cloud.google.com/bigquery/docs/access-control#bigquery.resourceViewer) -> This role is for Table-Level Lineage and Usage extraction | ||
* [Logs View Accessor](https://cloud.google.com/bigquery/docs/access-control#bigquery.dataViewer) -> This role is for Table-Level Lineage and Usage extraction | ||
* [BigQuery Data Viewer](https://cloud.google.com/bigquery/docs/access-control#bigquery.dataViewer) -> This role is for Profiling | ||
* [BigQuery Read Session User](https://cloud.google.com/bigquery/docs/access-control#bigquery.readSessionUser) -> This role is for Profiling |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice -- thank you for catching this!
--- | ||
# Configuring Your Redshift Connector to DataHub | ||
|
||
Now that you have created a DataHub user in Redshift in [the prior step](setup.md), it's now time to set up a connection via the DataHub UI. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
change "it's now time to set up" to "it's time to set up" -- no now required
|
||
* **Usage statistics** to help you understand recent query activity | ||
* **Table-level lineage** (where available) to automatically define interdependencies between datasets | ||
* **Table- and column-level profile statistics** to help you understand the shape of the data |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"to help you understand relationships between Tables and Columns"
## Redshift Prerequisites | ||
|
||
1. Connect to your Amazon Redshift cluster using an SQL client such as SQL Workbench/J or Amazon Redshift Query Editor with your Admin user. | ||
2. Create a [User](https://docs.aws.amazon.com/redshift/latest/gsg/t_adding_redshift_user_cmd.html) if you don't have one already. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"Create a Redshift User" that will be used to perform the metadata extraction
@@ -52,6 +52,9 @@ In order to configure ingestion from Snowflake, you'll first have to ensure you | |||
grant references on all views in database identifier($db_var) to role datahub_role; | |||
grant references on future views in database identifier($db_var) to role datahub_role; | |||
|
|||
-- Assign privileges to extract lineage and usage statistics from Snowflake by executing the below query. | |||
grant imported privileges on database snowflake to role datahub_role; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thank you!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor comments overall looks great! Merged your static assets PR.
Adding Redshift doc to sidebar.js
Adding Redshift ingestion quickstart guide
Small fix for BigQuery and Snowflake doc
Checklist