Skip to content

Add 2 new TableTypes#632

Open
sandormatyas wants to merge 3 commits intodatabricks:mainfrom
sandormatyas:chore/add-table-types
Open

Add 2 new TableTypes#632
sandormatyas wants to merge 3 commits intodatabricks:mainfrom
sandormatyas:chore/add-table-types

Conversation

@sandormatyas
Copy link

@sandormatyas sandormatyas commented Apr 29, 2024

Changes

(rebased, expanded #476)

Adds two new values to TableType Enum:

Unfortunately, these are not mentioned by the API documentation for the table_type field: https://docs.databricks.com/api/workspace/tables/list#tables-table_type

Here's an API call that fetches some tables, one with type MANAGED_SHALLOW_CLONE:
image

Here's an API call that fetches some tables, one with type FOREIGN:
image

Update:
Online tables also come with type FOREIGN, but different data_source_format:
image

Tests

  • make test run locally
  • make fmt applied
  • relevant integration tests applied

Signed-off-by: Matyas Sandor <sandormatyas7@gmail.com>
Signed-off-by: Matyas Sandor <sandormatyas7@gmail.com>
@sandormatyas sandormatyas force-pushed the chore/add-table-types branch from d28002e to 732d33b Compare April 29, 2024 15:11
@sandormatyas
Copy link
Author

Hi @mgyucht, sorry for tagging you, but I was wondering if I could get some eyes on this or tips on how I could proceed?

We are fetching the table metadata using the sdk and we have multiple None-s now for table type.

@MrTeale
Copy link

MrTeale commented Nov 20, 2024

Any chance of getting this progressed? FOREIGN not being a TableType is causing some issues for me

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