Skip to content

Remove experimentalIsUnifiedHost flag from DatabricksConfig#773

Merged
hectorcast-db merged 2 commits intomainfrom
hector/remove-experimental-unified-host-flag
Apr 22, 2026
Merged

Remove experimentalIsUnifiedHost flag from DatabricksConfig#773
hectorcast-db merged 2 commits intomainfrom
hector/remove-experimental-unified-host-flag

Conversation

@hectorcast-db
Copy link
Copy Markdown
Contributor

Summary

Ports databricks/databricks-sdk-go#1641 and databricks/databricks-sdk-py#1358 to the Java SDK.

Drops the experimentalIsUnifiedHost field and the DATABRICKS_EXPERIMENTAL_IS_UNIFIED_HOST environment variable from DatabricksConfig. The flag was never read in production:

  • DatabricksConfig.getHostType() derives the host type purely from the URL pattern.
  • Unified-host metadata fields (accountId, workspaceId, discoveryUrl) are auto-resolved via /.well-known/databricks-config during config.resolve().

No production path referenced the flag, and no tests set it (confirmed via grep across the repo).

Breaking change

This is a compile-time breaking change. Downstream callers that were setting the flag should drop those calls:

// Remove any usage of:
config.setExperimentalIsUnifiedHost(true);
config.getExperimentalIsUnifiedHost();

// Remove the env var from deployment configs:
DATABRICKS_EXPERIMENTAL_IS_UNIFIED_HOST

No user-facing replacement is needed — unified host detection is automatic.

Scope vs. the Go/Python PRs

Change Applies to Java?
Remove Experimental_IsUnifiedHost field + env var ✅ ported here
Remove ErrWorkspaceIDInAccountClient (Go only) ❌ no equivalent in Java
README docs for unified host / default auth flow ➖ skipped — Java's README does not cover the same sections; documentation can follow in a separate PR if desired

Test plan

  • mvn compile clean
  • mvn test — 1187 tests pass, no regressions
  • Spotless clean
  • No remaining references to experimentalIsUnifiedHost / EXPERIMENTAL_IS_UNIFIED_HOST in the repo (grep clean)

This pull request was AI-assisted by Isaac.

Drops the experimentalIsUnifiedHost field and the
DATABRICKS_EXPERIMENTAL_IS_UNIFIED_HOST env var. The flag was never
read — getHostType() derives the host type from the URL pattern, and
unified-host metadata fields (accountId, workspaceId, discoveryUrl)
are auto-resolved via /.well-known/databricks-config during
config.resolve(). No production path referenced the flag, and no
tests set it.

Ports databricks/databricks-sdk-go#1641 and
databricks/databricks-sdk-py#1358.

This is a compile-time breaking change. Downstream callers that were
setting the flag should drop those calls.

Co-authored-by: Isaac
@hectorcast-db hectorcast-db force-pushed the hector/remove-experimental-unified-host-flag branch from db1c61c to 3e81d21 Compare April 22, 2026 08:55
@hectorcast-db hectorcast-db requested a review from tanmay-db April 22, 2026 08:55
Completes the port of databricks/databricks-sdk-go#1641 and
databricks/databricks-sdk-py#1358 by aligning README with what Go and
Python published. Changes:

- Add "Unified host support" entry + "GCP native authentication" entry
  to the auth "In this section" TOC.
- List GCP as the third step of the default auth order, and explain that
  auth methods auto-skip when their required config is absent. Show how
  to force a method via DatabricksConfig.setAuthType.
- Add a "Unified host support" section with a .databrickscfg example and
  equivalent Java code (WorkspaceClient + AccountClient sharing one
  profile, plus overriding workspace_id).
- Extend the native-auth attribute table with workspace_id and
  discovery_url, mark account_id / workspace_id / discovery_url as
  auto-discoverable, and note that auto-discovery never overwrites
  explicit values.

The Go/Python PRs also document a "cloud" config field; Java does not
(yet) expose one on DatabricksConfig, so it is omitted here.

Co-authored-by: Isaac
@github-actions
Copy link
Copy Markdown
Contributor

If integration tests don't run automatically, an authorized user can run them manually by following the instructions below:

Trigger:
go/deco-tests-run/sdk-java

Inputs:

  • PR number: 773
  • Commit SHA: 1ffda2c7cd35648cffa45784893bc74040b07d8d

Checks will be approved automatically on success.

@hectorcast-db hectorcast-db added this pull request to the merge queue Apr 22, 2026
Merged via the queue into main with commit 86798d3 Apr 22, 2026
16 checks passed
@hectorcast-db hectorcast-db deleted the hector/remove-experimental-unified-host-flag branch April 22, 2026 10:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants