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.

Sign up for GitHub

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

Jump to bottom

Tableau ingestion didn't avoid unpublished sheets. #6421

Closed
YuriyGavrilov opened this issue Nov 14, 2022 · 2 comments · Fixed by #11559
Closed

Tableau ingestion didn't avoid unpublished sheets. #6421

YuriyGavrilov opened this issue Nov 14, 2022 · 2 comments · Fixed by #11559
Assignees
@treff7es
Labels
accepted An Issue that is confirmed as a bug by the DataHub Maintainers. bug Bug report ingestion PR or Issue related to the ingestion of metadata

Comments

@YuriyGavrilov
Copy link
Contributor

Describe the bug

The Tableau has the functionality to stay the sheets unpublished, nevertheless they will be used on one of the dashboards. There could be several unpublished sheets which in use on one dashbord. To avoid dublicates there should be an option to avoid unpublished sheets.

To Reproduce
Steps to reproduce the behavior:

  1. Go to 'Tableau and create Dashboard based on few unpublished sheets.'
  2. Click on 'Tableau insetion RUN. '
  3. Go to the Ingestion results
  4. See all the sheets even it was marked as unpublished in Tabeau.

Expected behavior
Unpublished sheet in Tableau shoued be avoided in Datahub ingestion by default or as option.

Screenshots
Unpublish Tableau flag
we see all the sheets and dashboards even it flaget unpublished

Desktop (please complete the following information):

  • OS: [e.g. iOS]
  • Browser [e.g. chrome, safari]
  • Version [e.g. 22]

Additional context
we see in datahub all the sheets and dashboards even it flaget unpublished in Tableau.
@YuriyGavrilov YuriyGavrilov added the bug Bug report label Nov 14, 2022
@aditya-radhakrishnan aditya-radhakrishnan added the ingestion PR or Issue related to the ingestion of metadata label Nov 15, 2022
@hsheth2 hsheth2 added the accepted An Issue that is confirmed as a bug by the DataHub Maintainers. label Dec 6, 2022
@YuriyGavrilov YuriyGavrilov mentioned this issue Jan 17, 2023
Closed
@YuriyGavrilov
Copy link
Contributor Author

YuriyGavrilov commented May 27, 2024 •
edited
Loading

@hsheth2 @treff7es May be it is time to take it?

Today we could have 50 tableau sheet with the name sheet1, sheet2, sheet3 ... and some sheet as old version of one so only one will be published on server but others exist in workbook but hidden.

I counted 1369 sheet* on our server

There is also good to fix #10121 to have full level of workbooks to simply navigate workbook and corespond sheets, chats, datasets, data source ...

@aviv-julienjehannet
Copy link
Contributor

Tableau uses some weird logic for hidden dashboards and cannot be retrieved by GraphQL cleanly.
however, @haeniya is working on the feature already.

Related to #11559

@hsheth2 hsheth2 mentioned this issue Dec 5, 2024
Merged
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@treff7es treff7es

Labels
accepted An Issue that is confirmed as a bug by the DataHub Maintainers. bug Bug report ingestion PR or Issue related to the ingestion of metadata
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(ingestion/tableau): hidden asset handling haeniya/datahub
5 participants
@treff7es @hsheth2 @aditya-radhakrishnan @YuriyGavrilov @aviv-julienjehannet