Skip to content

Commit 841426e

Browse files
committed
add validation where there was none
1 parent e2132d8 commit 841426e

File tree

3 files changed

+34
-0
lines changed

3 files changed

+34
-0
lines changed

CHANGELOG.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,10 @@
99
* Add `experimentalIsUnifiedHost` field to `DatabricksConfig` to mark unified hosts
1010
* Add `getHostType()` and `getConfigType()` methods to `DatabricksConfig`
1111
* Add X-Databricks-Org-Id header support for unified host workspace requests
12+
* Improve validation in `AccountClient` and `WorkspaceClient` constructors:
13+
* `AccountClient` now validates that `accountId` is set and `workspaceId` is not set
14+
* `WorkspaceClient` now validates that host is not an account host
15+
* `WorkspaceClient` with unified host now requires `workspaceId` to be set
1216

1317
### Deprecations
1418
* Deprecate `isAccountClient()` method in `DatabricksConfig`. Use `getHostType()` or `getConfigType()` instead.

databricks-sdk-java/src/main/java/com/databricks/sdk/AccountClient.java

Lines changed: 17 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

databricks-sdk-java/src/main/java/com/databricks/sdk/WorkspaceClient.java

Lines changed: 13 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)