Skip to content

fix: Document Databricks SQL Warehouse tuning parameters and client_timeout#1510

Merged
lukekim merged 3 commits intotrunkfrom
fix/docs-databricks-sql-warehouse-tuning-params
Apr 15, 2026
Merged

fix: Document Databricks SQL Warehouse tuning parameters and client_timeout#1510
lukekim merged 3 commits intotrunkfrom
fix/docs-databricks-sql-warehouse-tuning-params

Conversation

@claudespice
Copy link
Copy Markdown
Collaborator

Summary

  • The Databricks catalog connector has 5 SQL Warehouse connection tuning parameters (max_concurrent_requests, http_max_retries, backoff_method, statement_max_retries, disable_on_permanent_error) and a client_timeout parameter that were implemented in code but not documented.
  • The SQL Warehouse tuning parameters only exist in trunk (vNext), so they are added only to the unversioned docs.
  • The client_timeout parameter exists in both trunk and v1.11.5, so it is added to both vNext and v1.11.x versioned docs.

Changes

  • Added "SQL Warehouse connection tuning" table to vNext Databricks catalog docs with all 5 tuning parameters and their defaults
  • Added client_timeout parameter documentation to both vNext and v1.11.x Databricks catalog docs

Reference

Verified against spiceai/spiceai at trunkcrates/runtime/src/catalogconnector/databricks.rs lines 80-104
Verified client_timeout exists at v1.11.5 — same file, line 78

…imeout

Add documentation for undocumented Databricks catalog connector parameters:

- SQL Warehouse connection tuning params (vNext only): max_concurrent_requests,
  http_max_retries, backoff_method, statement_max_retries,
  disable_on_permanent_error
- client_timeout parameter for Delta Lake object store client (both vNext and v1.11.x)
@claudespice claudespice added the bug Something isn't working label Apr 14, 2026
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 14, 2026

✅ Pull with Spice Passed

Passing checks:

  • ✅ Title meets minimum length requirement (10 characters)
  • ✅ Has at least one of the required labels: area/blog, area/docs, area/cookbook, dependencies
  • ✅ No banned labels detected
  • ✅ Has at least one assignee: lukekim

@github-actions
Copy link
Copy Markdown

🔍 Pull with Spice Failed

Passing checks:

  • ✅ Title meets minimum length requirement (10 characters)
  • ✅ No banned labels detected

Failed checks:

  • ❌ Please select at least one of the required labels for this pull request: area/blog, area/docs, area/cookbook, dependencies
  • ❌ At least one assignee is required for this pull request.

Please address these issues and update your pull request.

@github-actions
Copy link
Copy Markdown

🚀 deployed to https://34d5bc9b.spiceai-org-website.pages.dev

@github-actions
Copy link
Copy Markdown

🚀 deployed to https://d55900b0.spiceai-org-website.pages.dev

@github-actions
Copy link
Copy Markdown

🚀 deployed to https://936fa5e4.spiceai-org-website.pages.dev

@lukekim lukekim merged commit 27f1e6f into trunk Apr 15, 2026
6 of 7 checks passed
@lukekim lukekim deleted the fix/docs-databricks-sql-warehouse-tuning-params branch April 15, 2026 02:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/docs bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants