Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Add Connection, NamedConnection and Relation classes#204

Closed
kgpayne wants to merge 3 commits intotableau:developmenttableau/document-api-python:developmentfrom
tailsdotcom:mastertailsdotcom/document-api-python:masterCopy head branch name to clipboard
Closed

Add Connection, NamedConnection and Relation classes#204
kgpayne wants to merge 3 commits intotableau:developmenttableau/document-api-python:developmentfrom
tailsdotcom:mastertailsdotcom/document-api-python:masterCopy head branch name to clipboard

Conversation

@kgpayne
Copy link

@kgpayne kgpayne commented Jun 29, 2021 •
edited
Loading

New Connection, NamedConnection and Relation classes, added to support extraction of metadata via tap-tableau-server. From the readme:

tap-tableau-server is a Singer tap for Tableau Server, focused (currently) on extracting details embedded inside Workbook files. This includes Datasources, Connections and Relations, as well as retrieving table references from embedded Custom SQL text fields inside Relation entities.

Extracting the specifics of Datasources, Connections, Relations and Table References from each Workbook help answer questions like:

  • Which Workbooks depend on which tables in which databases?
  • How many Workbooks depend on Excel, CSV or Google Sheets?
  • Who's credentials are used for embedded connections, in which Workbooks?

Having answers to all of these questions has helped us with wrangling our Tableau Server instance, which is several years old and has over 1000 Workbooks.

In future, we hope to extend this tap to cover other metadata that is exposed by the Tableau Server API's directly (esp. Published Connections and child objects). PR submission very welcome. Watch this space!

kgpayne added 3 commits February 7, 2021 10:27
Added new entities and a to_dict() methods. (#1)
d831833 
* Added new relation entity and to_dict() methods.

* More tidying.
Minor tweaks. (#2)
fc12713 
* Added new relation entity and to_dict() methods.

* More tidying.

* Minor tweaks.
Merge branch 'tableau:master' into master
b9ad9bc
@kgpayne kgpayne changed the title Add Connection Add Connection, NamedConnection and Relation classes Jun 29, 2021
@jacalata
Copy link
Contributor

jacalata commented Sep 16, 2021

Interesting! Could you please submit a Contributor License Agreement described here: https://tableau.github.io/contributing.html?

@bryceglarsen
Copy link

bryceglarsen commented Oct 29, 2021

Any update on this? Would love to be able to use the new relation class.

@jacalata
Copy link
Contributor

jacalata commented Oct 30, 2021

Unfortunately we absolutely can't merge code from someone who hasn't agreed to our CLA. Hopefully @kgpayne is still available and might come back to sign it and submit!

@jacalata
Copy link
Contributor

jacalata commented Oct 30, 2021

But I can switch the PR to branch from development instead of master, and I'll try and add some tests for it so it's easier to finish if they do come back.

@jacalata jacalata changed the base branch from master to development October 30, 2021 19:49
@bryceglarsen
Copy link

bryceglarsen commented Oct 31, 2021

Completely understandable @jacalata - wasn’t sure if CLA had been signed. Thank you though!

@kgpayne
Copy link
Author

kgpayne commented Nov 2, 2021

Hey 👋 Thanks for checking this out! I recently moved roles, and am no longer at tailsdotcom/ to be able to sign an Organisational CLA on their behalf. However @alanmcruickshank might be able to 🙂 He is familiar with this work. I also don't have access to a Tableau instance anymore, so really appreciate your work on tests @jacalata 🙏

@jacalata jacalata deleted the branch tableau:development December 16, 2021 08:21
@jacalata jacalata closed this Dec 16, 2021
@jacalata jacalata reopened this Dec 16, 2021
@salesforce-cla
Copy link

salesforce-cla bot commented Dec 16, 2021

Thanks for the contribution! Before we can merge this, we need @kgpayne to sign the Salesforce.com Contributor License Agreement.

@jacalata
Copy link
Contributor

jacalata commented Mar 14, 2022

Hi @alanmcruickshank - are you interested/able to sign a CLA so I could merge this work into the tool? I don't think the bot is smart enough to understand that, so we might just need it over email.

@alanmcruickshank
Copy link

alanmcruickshank commented Mar 21, 2022

@jacalata - sure thing. Devraj Toor is our new tableau account rep so he can probably facilitate some document signing and has my email address 👍 .

@jacalata
Copy link
Contributor

jacalata commented May 24, 2022

I finally actually pinged Dev so hopefully will get this wrapped up soon

@waltonereed
Copy link

waltonereed commented Aug 4, 2022

Any updates here? Am interested in the relation class too.

@kgpayne
Copy link
Author

kgpayne commented Feb 8, 2023 •
edited
Loading

Closing as approaching 2 years old and very stale 🤷‍♂️

@kgpayne kgpayne closed this Feb 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

cla:missing CLARequired enhancement

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@kgpayne @jacalata @bryceglarsen @alanmcruickshank @waltonereed
Morty Proxy This is a proxified and sanitized view of the page, visit original site.