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.

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

fix array listing in Tableau #10

Merged
merged 1 commit into from
Sep 15, 2023

Conversation

DimitrisStaratzis
Copy link
Contributor

@DimitrisStaratzis DimitrisStaratzis commented Sep 15, 2023

This PR fixes the array listing in Tableau.

It also reverts to only using the array names in the listing ignoring the fact that there might be arrays with duplicate names.
The previous version was displaying each array with a name of the form:
tiledb://namespace/name ~ tiledb://namespace/uuid.

This however was causing problems in Tableau because:

  1. it was modifying the name and
  2. it was shortening the name.

and this made it difficult for us to collect the info we wanted which is the tiledbURI.

An example of the logs can be seen here:

2023-09-15 13:08:18.304 +0300 (,,,,9,93) grpc-default-executor-3 : INFO  com.tableau.connect.service.ProtocolService - Running query 
   SELECT `tiledb___shaunreed_dense-array-char-dimensions ~ tiledb___shau`.`a1` AS `a1`,
     `tiledb___shaunreed_dense-array-char-dimensions ~ tiledb___shau`.`cols` AS `cols`,
     `tiledb___shaunreed_dense-array-char-dimensions ~ tiledb___shau`.`rows` AS `rows`
   FROM `tiledb://shaunreed/dense-array-char-dimensions ~ tiledb://shaunreed/e8a09831-53a2-4477-9f3e-bd2b82a809ec`.`tiledb://shaunreed/dense-array-char-dimensions ~ tiledb://shaunreed/e8a09831-53a2-4477-9f3e-bd2b82a809ec` `tiledb___shaunreed_dense-array-char-dimensions ~ tiledb___shau`
   LIMIT 100

As you can see Tableau modifies the name in various ways and I deemed appropriate to step away from trying to make this work as I think we will just introduce more errors and edge cases.

Now, if a user tries to view the data from an array with a duplicate name he/she will receive an error message. He/She can then proceed to query the specific array from the "Custom SQL" field in both Tableau and Power BI by using the UUID. We could instead list all the arrays with their tiledbURI but this would look ugly.

This PR has been tested in Both Tableau and Power BI

Screenshots below:

Screenshot 2023-09-15 at 5 29 47 PM Screenshot 2023-09-15 at 5 32 23 PM Screenshot 2023-09-15 at 5 32 55 PM

@DimitrisStaratzis DimitrisStaratzis merged commit e6ff22e into main Sep 15, 2023
@DimitrisStaratzis DimitrisStaratzis deleted the dstara/fixing_tableau_not_listing_arrays branch September 15, 2023 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants