[Feature] Add support for Unified Host with experimental flag#1135
Merged
Conversation
renaudhartert-db
requested changes
Dec 11, 2025
8d86309 to
fa36540
Compare
|
If integration tests don't run automatically, an authorized user can run them manually by following the instructions below: Trigger: Inputs:
Checks will be approved automatically on success. |
renaudhartert-db
approved these changes
Dec 16, 2025
Contributor
Author
|
Verified external-browser auth flow works, for existing and spog use cases, browser is opened and the further API calls are successful. |
github-merge-queue Bot
pushed a commit
to databricks/databricks-sdk-java
that referenced
this pull request
Jan 21, 2026
## What changes are proposed in this pull request? This PR adds support for unified host: - Separates client type from host type determination, deprecating `isAccountClient` and replacing it with `getHostType()` and `getClientType()` methods using new `HostType` and `ClientType` enums - Adds an experimental flag to indicate if a host is unified: `experimentalIsUnifiedHost` - Adds a `workspaceId` attribute to DatabricksConfig, which is necessary for workspace clients that talk to unified hosts - Adds `getUnifiedOidcEndpoints()` function, which is used in the OIDC endpoint resolution logic to discover OAuth endpoints on unified hosts - Adds header injection logic in DatabricksConfig.authenticate() which adds an `X-Databricks-Org-Id` header to requests made by workspace clients on unified hosts - Adds comprehensive test coverage including unit tests for host/client type detection, OIDC endpoint resolution, header injection, and integration tests Similar to what is done for: - [databricks/databricks-sdk-py#1135](databricks/databricks-sdk-py#1135) - [databricks/databricks-sdk-go#1307](databricks/databricks-sdk-go#1307) ## How is this tested? - Unit tests - Manually E2E tested with a spog profile
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
What changes are proposed in this pull request?
This PR adds support for unified host:
enums
Similar to what is done for databricks/databricks-sdk-go#1307
How is this tested?
Note: Integration test would be added in another PR once the test infra supports spog