Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
  • Loading branch information
hsheth2 authored Jan 3, 2025
1 parent 04d6a83 commit eb81059
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions metadata-ingestion/docs/sources/tableau/tableau_pre.md
Original file line number Diff line number Diff line change
Expand Up @@ -14,13 +14,13 @@ DataHub supports two authentication methods:

Either way, the user/token must have the **Site Administrator Explorer** site role.

::tip
:::info

We need the `Site Administrator Explorer` site role in order to get complete metadata from Tableau.

With any lower role, the Tableau Metadata API returns missing/partial metadata. This particularly affects data source fields and definitions, which impacts our ability to extract columns and generate column lineage. As such, other site roles like `Viewer` are insufficient with the current Tableau Metadata API.

::
:::

### Ingestion through UI

Expand Down

0 comments on commit eb81059

Please sign in to comment.