feat: add support for alter schema drop vector index by antas-marcin · Pull Request #1991 · weaviate/weaviate-python-client

antas-marcin · 2026-03-23T13:56:03Z

Summary

Add delete_vector_index(vector_name) method to collection config, allowing users to drop a named vector's index via DELETE /v1/schema/{className}/vectors/{vectorIndexName}/index
Follows the same pattern as the existing delete_property_index method
Includes both sync and async type stubs

Closes #1990

orca-security-eu

Orca Security Scan Summary

Status	Check	Issues by priority
Passed	Infrastructure as Code	0 0 0 0	View in Orca
Passed	SAST	0 0 0 0	View in Orca
Passed	Secrets	0 0 0 0	View in Orca
Passed	Vulnerabilities	0 0 0 0	View in Orca

Copilot

Pull request overview

Adds client support for dropping a named vector’s index via the schema REST endpoint, mirroring the existing “delete property index” capability in the collections config API.

Changes:

Added delete_vector_index(vector_name) to the collection config executor, issuing DELETE /v1/schema/{className}/vectors/{vectorIndexName}/index.
Added sync and async type stubs for delete_vector_index(...).

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

File	Description
weaviate/collections/config/executor.py	Implements the new delete-vector-index operation via the config executor.
weaviate/collections/config/sync.pyi	Exposes the new method in the sync config type stub.
weaviate/collections/config/async_.pyi	Exposes the new method in the async config type stub.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

+        Raises:
+            weaviate.exceptions.WeaviateConnectionError: If the network connection to Weaviate fails.
+            weaviate.exceptions.UnexpectedStatusCodeError: If Weaviate reports a non-OK status.
+            weaviate.exceptions.WeaviateInvalidInputError: If the vector does not exist.


+        def resp(res: Response) -> bool:
+            return res.status_code == 200
+


+        return executor.execute(
+            response_callback=resp,
+            method=self._connection.delete,
+            path=path,
+            error_msg="Vector may not exist",
+            status_codes=_ExpectedStatusCodes(ok_in=[200], error="vector exists"),
+        )


+    def delete_vector_index(
+        self,
+        vector_name: str,
+    ) -> executor.Result[bool]:
+        """Delete a vector index from the collection in Weaviate.
+
+            This is a destructive operation. The index will
+            need to be regenerated if you wish to use it again.
+
+        Args:
+            vector_name: The name of the vector whose index to delete.
+
+        Raises:
+            weaviate.exceptions.WeaviateConnectionError: If the network connection to Weaviate fails.
+            weaviate.exceptions.UnexpectedStatusCodeError: If Weaviate reports a non-OK status.
+            weaviate.exceptions.WeaviateInvalidInputError: If the vector does not exist.
+        """
+        _validate_input(
+            [_ValidateArgument(expected=[str], name="vector_name", value=vector_name)]
+        )
+
+        path = (
+            f"/schema/{_capitalize_first_letter(self._name)}"
+            + f"/vectors/{vector_name}"
+            + "/index"
+        )
+
+        def resp(res: Response) -> bool:
+            return res.status_code == 200
+
+        return executor.execute(
+            response_callback=resp,
+            method=self._connection.delete,
+            path=path,
+            error_msg="Vector may not exist",
+            status_codes=_ExpectedStatusCodes(ok_in=[200], error="vector exists"),
+        )


orca-security-eu

Orca Security Scan Summary

Status	Check	Issues by priority
Passed	Secrets	0 0 0 0	View in Orca

Copilot

Pull request overview

Copilot reviewed 12 out of 12 changed files in this pull request and generated 2 comments.

            vector_index_config = __get_vector_index_config(named_vector)
-            assert vector_index_config is not None
+            # A vector whose index was dropped with `collection.config.delete_vector_index` is
+            # returned as `vectorIndexType: "none"` without any `vectorIndexConfig`.
+            assert (
+                vector_index_config is not None


+        NONE: The index of this vector has been dropped, see ``collection.config.delete_vector_index``.
+            The vector data is still stored, but it cannot be searched. This value is reported by the
+            server only, it cannot be used to configure a vector.
    """


Follow-up to the `delete_vector_index` support, addressing review findings. After a successful drop, Weaviate keeps the vector in the schema as `vectorIndexType: "none"` with no `vectorIndexConfig`. The client asserted that every named vector has an index config, so `collection.config.get()` and `client.collections.list_all()` raised `AssertionError` for every collection in the cluster once any vector index had been dropped. `_NamedVectorConfig. vector_index_config` is now optional, `VectorIndexType` gained a server-reported `NONE` member and `to_dict()` round-trips it. `collection.config.update()` on a dropped vector raised a bare `KeyError: 'vectorIndexConfig'` from the schema merge. Both the current and the deprecated merge paths now go through one helper that raises a `WeaviateInvalidInputError` explaining that a dropped index cannot be re-created. The docstring claimed the index could be regenerated and that a missing vector raises `WeaviateInvalidInputError`. Neither is true: Weaviate rejects re-creating a dropped index, and an unknown vector name comes back as a 422. It now also documents that the endpoint is experimental and needs `ENABLE_EXPERIMENTAL_ALTER_SCHEMA_DROP_VECTOR_INDEX_ENDPOINT=true`, that only named vectors can be dropped, and that the drop is applied asynchronously. The error message no longer blames a missing vector for what is usually a disabled endpoint. Tests: unit coverage for parsing, exporting and updating a dropped vector, mock coverage for the request path and the disabled-endpoint response, and integration coverage gated at 1.39.0. The CI compose file enables the experimental endpoint; that flag can be dropped once 1.39.0 is GA. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

bevzzz · 2026-07-22T15:38:44Z

thought: Do you think this PR could do with fewer tests? The new endpoint is very simple and it seems to me like one "happy path" test will catch most things.

Other tests verify Weaviate's behavior more than client's logic, and some may add up to 30s of pipeline time.

orca-security-eu Bot reviewed Mar 23, 2026

View reviewed changes

jfrancoa requested a review from Copilot July 22, 2026 08:52

Copilot started reviewing on behalf of jfrancoa July 22, 2026 08:53 View session

Copilot AI reviewed Jul 22, 2026

View reviewed changes

jfrancoa changed the base branch from main to bump-integration-tests-1.39 July 22, 2026 11:25

jfrancoa force-pushed the alter-schema-drop-vector-index branch from d728dd9 to 7e5cd7f Compare July 22, 2026 11:25

jfrancoa requested a review from a team as a code owner July 22, 2026 11:25

orca-security-eu Bot reviewed Jul 22, 2026

View reviewed changes

jfrancoa requested a review from Copilot July 22, 2026 11:48

Copilot started reviewing on behalf of jfrancoa July 22, 2026 11:48 View session

jfrancoa force-pushed the bump-integration-tests-1.39 branch from 3657487 to 736a7cd Compare July 22, 2026 11:50

Copilot AI reviewed Jul 22, 2026

View reviewed changes

jfrancoa force-pushed the alter-schema-drop-vector-index branch from 7e5cd7f to 4e2a580 Compare July 22, 2026 11:53

jfrancoa mentioned this pull request Jul 22, 2026

ci: run integration tests against Weaviate 1.39.0-rc.0 #2099

Merged

jfrancoa force-pushed the bump-integration-tests-1.39 branch from 736a7cd to be6c8dc Compare July 22, 2026 12:05

antas-marcin and others added 2 commits July 22, 2026 14:06

feat: add support for alter schema drop vector index

4df5c74

jfrancoa force-pushed the alter-schema-drop-vector-index branch from 4e2a580 to 2dbfaed Compare July 22, 2026 12:07

This was referenced Jul 23, 2026

Add --drop_vector_index to weaviate-cli update collection weaviate/weaviate-cli#170

Open

Add --drop_vector_index to update collection weaviate/weaviate-cli#171

Open

Base automatically changed from bump-integration-tests-1.39 to main July 24, 2026 06:22

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat: add support for alter schema drop vector index#1991

feat: add support for alter schema drop vector index#1991
antas-marcin wants to merge 2 commits into
mainfrom
alter-schema-drop-vector-index

antas-marcin commented Mar 23, 2026 •

edited

Loading

Uh oh!

orca-security-eu Bot left a comment

Uh oh!

Copilot AI left a comment

Uh oh!

orca-security-eu Bot left a comment •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

bevzzz commented Jul 22, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

		def resp(res: Response) -> bool:
		return res.status_code == 200

Uh oh!

Conversation

antas-marcin commented Mar 23, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Uh oh!

orca-security-eu Bot left a comment

Choose a reason for hiding this comment

Orca Security Scan Summary

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

orca-security-eu Bot left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Orca Security Scan Summary

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

bevzzz commented Jul 22, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

antas-marcin commented Mar 23, 2026 •

edited

Loading

orca-security-eu Bot left a comment •

edited

Loading