Skip to content

Add Connection, NamedConnection and Relation classes - #204

Closed
kgpayne wants to merge 3 commits into
tableau:developmentfrom
tailsdotcom:master
Closed

Add Connection, NamedConnection and Relation classes#204
kgpayne wants to merge 3 commits into
tableau:developmentfrom
tailsdotcom:master

Conversation

@kgpayne

@kgpayne kgpayne commented Jun 29, 2021 •
edited
Loading

Copy link
Copy Markdown

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
kgpayne
Added new entities and a to_dict() methods. (#1)
d831833 
* Added new relation entity and to_dict() methods.

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

* More tidying.

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

jacalata commented Sep 16, 2021

Copy link
Copy Markdown
Contributor

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

@bryceglarsen

bryceglarsen commented Oct 29, 2021

Copy link
Copy Markdown

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

@jacalata

jacalata commented Oct 30, 2021

Copy link
Copy Markdown
Contributor

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

jacalata commented Oct 30, 2021

Copy link
Copy Markdown
Contributor

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

bryceglarsen commented Oct 31, 2021

Copy link
Copy Markdown

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

@kgpayne

kgpayne commented Nov 2, 2021

Copy link
Copy Markdown
Author

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

salesforce-cla Bot commented Dec 16, 2021

Copy link
Copy Markdown

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

@jacalata

jacalata commented Mar 14, 2022

Copy link
Copy Markdown
Contributor

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

alanmcruickshank commented Mar 21, 2022

Copy link
Copy Markdown

@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

jacalata commented May 24, 2022

Copy link
Copy Markdown
Contributor

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

@waltonereed

waltonereed commented Aug 4, 2022

Copy link
Copy Markdown

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

@kgpayne

kgpayne commented Feb 8, 2023 •
edited
Loading

Copy link
Copy Markdown
Author

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

See More

Successfully merging this pull request may close these issues.

5 participants

@kgpayne @jacalata @bryceglarsen @alanmcruickshank @waltonereed