dacort / metabase-athena-driver

An Amazon Athena driver for Metabase 0.32 and later
Apache License 2.0
225 stars 32 forks source link

Get `MANAGED_TABLE`s #93

Closed jpugliesi closed 2 years ago

jpugliesi commented 2 years ago

Athena is capable of querying managed tables in glue - this adds MANAGED_TABLE to the list of retrievable tables

Fixes dacort/metabase-athena-driver#91

dacort commented 2 years ago

Thanks @jpugliesi ! Have you had a chance to test this?

If not, give me a few days to quickly create a managed table and make sure it works. Otherwise, looks good. :)

jpugliesi commented 2 years ago

@dacort I have not had a chance to test manually - would appreciate if you can give it a run 🙏

jpugliesi commented 2 years ago

Any updates on this @dacort ?

dacort commented 2 years ago

@jpugliesi Hey there! Apologies for a little delayed with re:Invent. :) I'm on vacation this week so can hopefully check this out a little more. I'll update if I run into any issues.

dacort commented 2 years ago

Going to go ahead and merge this and cut a new release. :)

I haven't had a chance to test this on my own, but it's a fairly minor change and the only result should be these tables should now show up if folks have them. :)