Commit d3a4f93
authored
Migrate from databricks-vectorsearch to databricks-ai-search (#437)
## Summary
`databricks-vectorsearch` is deprecated and has been renamed to `databricks-ai-search`. Follow-up to #435.
When tests run, `databricks-vectorsearch` now emits:
> DeprecationWarning: databricks-vectorsearch is deprecated and has been renamed to databricks-ai-search. Imports under 'databricks.vector_search.*' will continue to work as a thin re-export of 'databricks.ai_search.*', but new code should switch to 'pip install databricks-ai-search' and 'from databricks.ai_search.* import ...'.
This PR follows that recommendation literally.
## Changes
- **`pyproject.toml` × 4** (root, `integrations/openai`, `integrations/langchain`, `integrations/llamaindex`): replace `databricks-vectorsearch>=*` with `databricks-ai-search>=0.73`.
- **12 Python files** under `src/`, `integrations/*/src/`, `integrations/*/tests/`, and `tests/`: every `from databricks.vector_search.* import ...` → `from databricks.ai_search.* import ...`. Includes `mock.patch` target strings and the `ImportError` fallback message in `databricks_langchain/vectorstores.py`.
- Symbol names are unchanged. `databricks-ai-search==0.73` exposes back-compat aliases for the names the bridge uses:
- `databricks/ai_search/client.py:1090`: `VectorSearchClient = AISearchClient`
- `databricks/ai_search/index.py:682`: `VectorSearchIndex = AISearchIndex`
- `databricks/ai_search/exceptions.py:47`: `VectorSearchException = AISearchException`
- `Reranker`, `DatabricksReranker`, `CredentialStrategy` exist under the same submodule paths.
Net diff: **+54 / −54 across 16 files** (pure rename).
## Validation
- **Package landscape**: in a fresh venv with the migrated branch installed editable, only `databricks-ai-search==0.73` is present — `databricks-vectorsearch` is no longer pulled at all.
- **Smoke**: `from databricks_openai.vector_search_retriever_tool import VectorSearchRetrieverTool` resolves `VectorSearchIndex` to `databricks.ai_search.index` even with the `databricks.vector_search` package completely absent.
- **Tests**:
- 53/53 openai unit tests pass.
- 258/258 langchain unit tests pass (4 pre-existing skips).
- `ruff check` + `ruff format --check` clean.
## Companion notes
- This complements #435 (the `VectorSearchIndex` canonical-path fix). With both changes shipped, the bridge no longer touches the `databricks.vector_search.*` namespace at all and stops triggering the deprecation warning.
- The langchain pyproject's `databricks-openai = { path = "../openai", editable = true }` source pin (added in #435) is preserved — still needed until the next `databricks-openai` release containing the canonical-path fix ships to PyPI; can be removed in a follow-up cleanup after that release.
- Out of scope: the cross-version matrix (`openai_test (3.10, v0.x)` / `langchain_test (3.10, v0.x)`) still tests *historical* tags whose frozen source imports `databricks.vector_search.client.VectorSearchIndex`. Those tags cannot be retroactively migrated — this PR doesn't try to. Tracked separately for Dhruv.
## Test plan
- [ ] CI green on this branch
- [ ] After merge + release: any consumers of `databricks_ai_bridge`, `databricks_langchain`, `databricks_openai`, or `databricks_llamaindex` no longer see the upstream `DeprecationWarning` about `databricks-vectorsearch`1 parent 4503aeb commit d3a4f93
16 files changed
Lines changed: 48 additions & 48 deletions
File tree
- integrations
- dspy/src/databricks_dspy/retrievers
- langchain
- src/databricks_langchain
- tests/unit_tests
- llamaindex
- src/databricks_llamaindex
- tests/unit_tests
- openai
- src/databricks_openai
- tests/unit_tests
- src/databricks_ai_bridge
- test_utils
- tests/integration_tests/vector_search
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
39 | 39 | | |
40 | 40 | | |
41 | 41 | | |
42 | | - | |
| 42 | + | |
43 | 43 | | |
44 | 44 | | |
45 | 45 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
10 | 10 | | |
11 | 11 | | |
12 | 12 | | |
13 | | - | |
| 13 | + | |
14 | 14 | | |
15 | 15 | | |
16 | 16 | | |
| |||
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | 3 | | |
4 | | - | |
| 4 | + | |
5 | 5 | | |
6 | 6 | | |
7 | 7 | | |
| |||
24 | 24 | | |
25 | 25 | | |
26 | 26 | | |
27 | | - | |
| 27 | + | |
28 | 28 | | |
29 | 29 | | |
30 | 30 | | |
| |||
Lines changed: 11 additions & 11 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
17 | 17 | | |
18 | 18 | | |
19 | 19 | | |
| 20 | + | |
20 | 21 | | |
21 | | - | |
22 | 22 | | |
23 | 23 | | |
24 | 24 | | |
| |||
67 | 67 | | |
68 | 68 | | |
69 | 69 | | |
70 | | - | |
| 70 | + | |
71 | 71 | | |
72 | 72 | | |
73 | 73 | | |
| |||
107 | 107 | | |
108 | 108 | | |
109 | 109 | | |
110 | | - | |
| 110 | + | |
111 | 111 | | |
112 | 112 | | |
113 | 113 | | |
| |||
249 | 249 | | |
250 | 250 | | |
251 | 251 | | |
252 | | - | |
253 | | - | |
| 252 | + | |
| 253 | + | |
254 | 254 | | |
255 | 255 | | |
256 | | - | |
257 | | - | |
| 256 | + | |
| 257 | + | |
258 | 258 | | |
259 | 259 | | |
260 | 260 | | |
| |||
422 | 422 | | |
423 | 423 | | |
424 | 424 | | |
425 | | - | |
| 425 | + | |
426 | 426 | | |
427 | 427 | | |
428 | 428 | | |
| |||
464 | 464 | | |
465 | 465 | | |
466 | 466 | | |
467 | | - | |
| 467 | + | |
468 | 468 | | |
469 | 469 | | |
470 | 470 | | |
| |||
538 | 538 | | |
539 | 539 | | |
540 | 540 | | |
541 | | - | |
| 541 | + | |
542 | 542 | | |
543 | 543 | | |
544 | 544 | | |
| |||
591 | 591 | | |
592 | 592 | | |
593 | 593 | | |
594 | | - | |
| 594 | + | |
595 | 595 | | |
596 | 596 | | |
597 | 597 | | |
| |||
Lines changed: 5 additions & 5 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
| 9 | + | |
9 | 10 | | |
10 | 11 | | |
11 | | - | |
12 | 12 | | |
13 | 13 | | |
14 | 14 | | |
| |||
301 | 301 | | |
302 | 302 | | |
303 | 303 | | |
304 | | - | |
| 304 | + | |
305 | 305 | | |
306 | 306 | | |
307 | 307 | | |
| |||
317 | 317 | | |
318 | 318 | | |
319 | 319 | | |
320 | | - | |
| 320 | + | |
321 | 321 | | |
322 | 322 | | |
323 | 323 | | |
| |||
332 | 332 | | |
333 | 333 | | |
334 | 334 | | |
335 | | - | |
| 335 | + | |
336 | 336 | | |
337 | 337 | | |
338 | 338 | | |
| |||
356 | 356 | | |
357 | 357 | | |
358 | 358 | | |
359 | | - | |
| 359 | + | |
360 | 360 | | |
361 | 361 | | |
362 | 362 | | |
| |||
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
7 | | - | |
8 | | - | |
| 7 | + | |
| 8 | + | |
9 | 9 | | |
10 | 10 | | |
11 | 11 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
9 | 9 | | |
10 | 10 | | |
11 | 11 | | |
12 | | - | |
| 12 | + | |
13 | 13 | | |
14 | 14 | | |
15 | 15 | | |
| |||
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
47 | 47 | | |
48 | 48 | | |
49 | 49 | | |
50 | | - | |
51 | | - | |
| 50 | + | |
| 51 | + | |
52 | 52 | | |
53 | 53 | | |
54 | 54 | | |
| |||
Lines changed: 6 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
7 | 10 | | |
8 | 11 | | |
9 | | - | |
10 | | - | |
11 | | - | |
12 | 12 | | |
13 | 13 | | |
14 | 14 | | |
| |||
156 | 156 | | |
157 | 157 | | |
158 | 158 | | |
159 | | - | |
| 159 | + | |
160 | 160 | | |
161 | 161 | | |
162 | 162 | | |
| |||
172 | 172 | | |
173 | 173 | | |
174 | 174 | | |
175 | | - | |
| 175 | + | |
176 | 176 | | |
177 | 177 | | |
178 | 178 | | |
| |||
182 | 182 | | |
183 | 183 | | |
184 | 184 | | |
185 | | - | |
| 185 | + | |
186 | 186 | | |
187 | 187 | | |
188 | 188 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
9 | 9 | | |
10 | 10 | | |
11 | 11 | | |
12 | | - | |
| 12 | + | |
13 | 13 | | |
14 | 14 | | |
15 | 15 | | |
| |||
0 commit comments