-
Notifications
You must be signed in to change notification settings - Fork 384
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
Add warehouse_type
parameter to sql_endpoint
to support DBSQL PRO SKU
#1728
Conversation
7b1ff43
to
9c1008c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pending response from @yuchenhuo
9c1008c
to
3ce49e2
Compare
sql_endpoint
: warehouse type & disable UCwarehouse_type
parameter to sql_endpoint
to support PRO SKU
warehouse_type
parameter to sql_endpoint
to support PRO SKUwarehouse_type
parameter to sql_endpoint
to support DBSQL PRO SKU
@nfx I reworked the change according to decision by dev team |
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #1728 +/- ##
=======================================
Coverage 90.00% 90.00%
=======================================
Files 141 141
Lines 11096 11098 +2
=======================================
+ Hits 9987 9989 +2
Misses 707 707
Partials 402 402
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
also need to add disable_uc
@@ -39,6 +39,7 @@ The following arguments are supported: | |||
* `enable_serverless_compute` - Whether this SQL endpoint is a Serverless endpoint. To use a Serverless SQL endpoint, you must enable Serverless SQL endpoints for the workspace. | |||
* `channel` block, consisting of following fields: | |||
* `name` - Name of the Databricks SQL release channel. Possible values are: `CHANNEL_NAME_PREVIEW` and `CHANNEL_NAME_CURRENT`. Default is `CHANNEL_NAME_CURRENT`. | |||
* `warehouse_type` - [SQL Warehouse Type](https://docs.databricks.com/sql/admin/sql-endpoints.html#switch-the-sql-warehouse-type-pro-classic-or-serverless): `PRO` (default) or `CLASSIC`. If Serverless SQL is enabled, you can only specify `PRO`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If serverless SQL is enabled, you can only specify PRO
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we don't know if serverless is enabled or not from the terraform execution, especially for non-admin
@carlrabelerDB it was discussed and decided not to do |
No description provided.