Skip to content

Deprecate graphql_raw_query - #2215

Open
g-despot wants to merge 1 commit into
mainfrom
feat/deprecate-graphql-raw-query
Open

g-despot wants to merge 1 commit into
mainfrom
feat/deprecate-graphql-raw-query

Conversation

@g-despot

Copy link
Copy Markdown
Collaborator

Emits a deprecation warning (Dep031) when graphql_raw_query is called. We're retiring GraphQL, so the method will be removed in a release after May 1, 2027. The warning points to the collection query methods, the REST Search API and the migration guide.

@orca-security-eu orca-security-eu Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Orca Security Scan Summary

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

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The focused changes follow existing warning conventions, preserve query execution, and have no identified blocking issues.

No open issues 🎉

What changed in this PR

Deprecates graphql_raw_query() ahead of GraphQL retirement, with removal planned for a release after May 1, 2027.

Changes:

  • Emits Dep031 with replacement APIs and a migration guide.
  • Documents the deprecation in the method docstring and changelog.
File Description
weaviate/​warnings.py Defines the Dep031 deprecation warning.
weaviate/​client_executor.py Emits the warning and documents the deprecation.
docs/​changelog.rst Announces deprecation and replacement APIs.

Review effort: Balanced

This branch has not been deployed

No deployments
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