-
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
[Feature] Add databricks_query
resource instead of databricks_sql_query
#4103
Open
alexott
wants to merge
10
commits into
main
Choose a base branch
from
new-query-resource
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+637
−43
Commits on Oct 19, 2024
-
[Feature] Add
databricks_alert
resource to replace `databricks_sql_……alert` New resource use the [new Alerts API](https://docs.databricks.com/api/workspace/alerts/create) instead of legacy one that will be deprecated. New resource has slightly different set of parameters, so it was decided to create new resource and deprecate old one. This resource uses old TF SDK to be compatible with TF exporter (until #4050 is implemented).
Configuration menu - View commit details
-
Copy full SHA for 9107638 - Browse repository at this point
Copy the full SHA 9107638View commit details -
Configuration menu - View commit details
-
Copy full SHA for 9793b54 - Browse repository at this point
Copy the full SHA 9793b54View commit details -
Configuration menu - View commit details
-
Copy full SHA for a46b16f - Browse repository at this point
Copy the full SHA a46b16fView commit details -
[Feature] Add
databricks_query
resource instead of `databricks_sql_……query` This PR is built on top of #4051 that should be merged first. The new resource uses the new [Queries API](https://docs.databricks.com/api/workspace/queries/create) instead of the legacy one that will be deprecated. Since the new resource has a slightly different set of parameters, it was decided to create a new resource and deprecate the old one. This resource uses old TF SDK to be compatible with TF exporter (until #4050 is implemented). TODOs: - Add documentation - Need to discuss how to handle permissions - `sql_query` permissions look like working, but not sure if we should continue to use that API - Support in the exporter will be in a separate PR
Configuration menu - View commit details
-
Copy full SHA for 5f47bb8 - Browse repository at this point
Copy the full SHA 5f47bb8View commit details -
Configuration menu - View commit details
-
Copy full SHA for 60d0d5f - Browse repository at this point
Copy the full SHA 60d0d5fView commit details -
Configuration menu - View commit details
-
Copy full SHA for 9e641a0 - Browse repository at this point
Copy the full SHA 9e641a0View commit details -
Configuration menu - View commit details
-
Copy full SHA for e680e99 - Browse repository at this point
Copy the full SHA e680e99View commit details -
Configuration menu - View commit details
-
Copy full SHA for 11082ab - Browse repository at this point
Copy the full SHA 11082abView commit details -
Configuration menu - View commit details
-
Copy full SHA for 15967bf - Browse repository at this point
Copy the full SHA 15967bfView commit details -
Configuration menu - View commit details
-
Copy full SHA for 73f19a0 - Browse repository at this point
Copy the full SHA 73f19a0View commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.