-
Notifications
You must be signed in to change notification settings - Fork 386
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
[ISSUE] Error when creating UC Managed table : Missing cloud file system scheme #1151
Comments
cc: sridharitv |
I'm fully willing to work on a fix, but I don't have access to the API spec. In the code, only the "EXTERNAL" table scenario is tested: So the problem is around a missing parameter for the MANAGED tables |
@ebarault it's an empty storage location that is causing the error. try filling it in. can you elaborate your idea on |
hi @nfx it is meant to be this way for Managed table. The doc states:
And it is just the way the example is provided. I find it weird that we have to provide an empty string for an Managed table, the best should be to not provide it at all, but then the provider raises an error that this property is required. What is your point regarding your last comment ?
|
@ebarault this resource needs a bit more love 😂
I'm looking at telemetry and I don't see plenty of customers using databricks_table. So I wonder - what scenarios do people want to use this resource? Eg setting up entire warehouse with terraform or just some individual tables? |
Ah, great question @nfx, and we're still evaluating the response to it. We don't plan to use this module to manage the tables schemas, but to manage the rights on those tables. Possible workflow:
We're also looking for evolutions where the privileges could be inherited from the database (schema), but we do keep the use case where a common database would gather multiple tables with different access control requirements for different teams (no read access for all) Generally speaking we're looking for a way to centralize all the access control on all databases/tables in one place |
@ebarault So it looks like you need https://registry.terraform.io/providers/databrickslabs/databricks/latest/docs/resources/grants. It's working with tables defined outside of terraform as well. Perhaps I should tune documentation to refer to it better. And yes, data resources for listing tables are coming |
@nfx Hi, I'm already using the grants module for sure. For metastore, for schemas, and which to use it for tables... in combination with terraform, not out the blue, referencing the table names. I'm not looking for workarounds :-) I trully want to rely on terraform to create UC tables, the module exists, it just has a bug, is it hard to fix (I fill it's just few things) ? Good thing that a data source is coming to list tables, but it's not my main path, i mostly this as a workaround. |
Hi @nfx |
Configuration
Expected Behavior
Table is created
Actual Behavior
Error:
Steps to Reproduce
Please list the steps required to reproduce the issue, for example:
terraform apply
Terraform and provider versions
terraform v1.1.0
databricks provider v0.5.0
Debug Output
The text was updated successfully, but these errors were encountered: