Deprecate Unity Catalog function integration in AI Gateway by harupy · Pull Request #19457 · mlflow/mlflow

harupy · 2025-12-17T09:34:57Z

🛠 DevTools 🛠

Install mlflow from this PR

# mlflow
pip install git+https://github.com/mlflow/mlflow.git@refs/pull/19457/merge
# mlflow-skinny
pip install git+https://github.com/mlflow/mlflow.git@refs/pull/19457/merge#subdirectory=libs/skinny

For Databricks, use the following command:

%sh curl -LsSf https://raw.githubusercontent.com/mlflow/mlflow/HEAD/dev/install-skinny.sh | sh -s pull/19457/merge

Related Issues/PRs

#xxx

What changes are proposed in this pull request?

Deprecate the Unity Catalog function integration feature in the MLflow AI Gateway:

Add deprecation warning in the documentation
Emit FutureWarning when MLFLOW_ENABLE_UC_FUNCTIONS is enabled
Update the environment variable docstring to indicate deprecation

How is this PR tested?

Existing unit/integration tests
New unit/integration tests
Manual tests

Does this PR require documentation update?

Release Notes

Is this a user-facing change?

No. You can skip the rest of this section.
Yes. Give a description of this change to be included in the release notes for MLflow users.

Unity Catalog function integration via the MLflow AI Gateway is now deprecated and will be removed in a future release.

What component(s), interfaces, languages, and integrations does this PR affect?

Components

How should the PR be classified in the release notes? Choose one:

rn/none - No description will be included. The PR will be mentioned only by the PR number in the "Small Bugfixes and Documentation Updates" section
rn/breaking-change - The PR will be mentioned in the "Breaking Changes" section
rn/feature - A new user-facing feature worth mentioning in the release notes
rn/bug-fix - A user-facing bug fix worth mentioning in the release notes
rn/documentation - A user-facing documentation change worth mentioning in the release notes

Should this PR be included in the next patch release?

Yes should be selected for bug fixes, documentation updates, and other small changes. No should be selected for new features and larger changes. If you're unsure about the release classification of this PR, leave this unchecked to let the maintainers decide.

What is a minor/patch release?

Minor release: a release that increments the second part of the version number (e.g., 1.2.0 -> 1.3.0).
Bug fixes, doc updates and new features usually go into minor releases.
Patch release: a release that increments the third part of the version number (e.g., 1.2.0 -> 1.2.1).
Bug fixes and doc updates usually go into patch releases.

Yes (this PR will be cherry-picked and included in the next patch release)
No (this PR will be included in the next minor release)

🤖 Generated with Claude Code

Signed-off-by: harupy <17039389+harupy@users.noreply.github.com>

Copilot

Pull request overview

This PR deprecates the Unity Catalog function integration feature in the MLflow AI Gateway to prepare for its eventual removal.

Key Changes:

Added runtime deprecation warning that emits a FutureWarning when MLFLOW_ENABLE_UC_FUNCTIONS is enabled
Updated environment variable docstring to indicate the feature is deprecated
Added deprecation notice at the top of the Unity Catalog integration documentation page

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File	Description
mlflow/gateway/providers/openai.py	Added `warnings` import and `FutureWarning` emission in the `chat()` method when UC functions are enabled
mlflow/environment_variables.py	Updated docstring for `MLFLOW_ENABLE_UC_FUNCTIONS` to indicate deprecation status
docs/docs/genai/governance/unity-catalog.mdx	Added prominent deprecation warning box at the top of the documentation page

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

docs/docs/genai/governance/unity-catalog.mdx

mlflow/gateway/providers/openai.py

harupy · 2025-12-17T09:42:06Z

/resolve

🚀 Resolve workflow started

✅ Workflow completed successfully.

Claude Output

Perfect! I've successfully addressed all unresolved PR review comments for PR #19457. Here's what was done:

Summary

PR: Deprecate Unity Catalog function integration in AI Gateway (#19457)

Unresolved Comments Addressed: 2

Changes Made

Updated deprecation message in documentation (docs/docs/genai/governance/unity-catalog.mdx:5)
- Made the deprecation warning more specific and clear
- Changed from generic "This feature is deprecated" to "Unity Catalog function integration via the MLflow AI Gateway is deprecated and will be removed in a future release."
Added alternative guidance to deprecation warning (mlflow/gateway/providers/openai.py:561-567)
- Added link to Databricks Managed MCP as an alternative solution
- Link: https://docs.databricks.com/aws/en/generative-ai/mcp/managed-mcp

Commit Details

Commit: 86304ee
All pre-commit hooks passed successfully
Changes committed locally with DCO sign-off
Not pushed to remote (as per instructions)

The changes are ready for review and all feedback has been addressed.

serena-ruan

LGTM! Should we include this in 3.8.0?

github-actions · 2025-12-17T09:42:43Z

Documentation preview for c608602 is available at:

https://pr-19457--mlflow-docs-preview.netlify.app/docs/latest/

Changed Pages (1)

genai/governance/unity-catalog (modified)

More info

Ignore this comment if this PR does not change the documentation.
The preview is updated when a new commit is pushed to this PR.
This comment was created by this workflow run.
The documentation was built by this workflow run.

- Make deprecation message more specific in Unity Catalog docs - Add alternative guidance link to deprecation warning 🤖 Generated with Claude Code Co-Authored-By: Claude <noreply@anthropic.com> Signed-off-by: mlflow-app[bot] <mlflow-app[bot]@users.noreply.github.com>

) Signed-off-by: harupy <17039389+harupy@users.noreply.github.com> Signed-off-by: mlflow-app[bot] <mlflow-app[bot]@users.noreply.github.com> Co-authored-by: mlflow-app[bot] <mlflow-app[bot]@users.noreply.github.com> Co-authored-by: Claude <noreply@anthropic.com>

Signed-off-by: harupy <17039389+harupy@users.noreply.github.com> Signed-off-by: mlflow-app[bot] <mlflow-app[bot]@users.noreply.github.com> Co-authored-by: mlflow-app[bot] <mlflow-app[bot]@users.noreply.github.com> Co-authored-by: Claude <noreply@anthropic.com>

Deprecate Unity Catalog function integration in AI Gateway

46e4e2f

Signed-off-by: harupy <17039389+harupy@users.noreply.github.com>

Copilot AI review requested due to automatic review settings December 17, 2025 09:34

github-actions bot added area/docs Documentation issues rn/feature Mention under Features in Changelogs. labels Dec 17, 2025

Copilot started reviewing on behalf of harupy December 17, 2025 09:35 View session

harupy added the team-review Trigger a team review request label Dec 17, 2025

github-actions bot requested review from BenWilson2 and serena-ruan December 17, 2025 09:36

Copilot AI reviewed Dec 17, 2025

View reviewed changes

docs/docs/genai/governance/unity-catalog.mdx Outdated Show resolved Hide resolved

serena-ruan reviewed Dec 17, 2025

View reviewed changes

mlflow/gateway/providers/openai.py Outdated Show resolved Hide resolved

serena-ruan approved these changes Dec 17, 2025

View reviewed changes

github-actions bot added the v3.8.0 label Dec 17, 2025

Merge branch 'master' into deprecate-uc-functions

c608602

harupy added this pull request to the merge queue Dec 17, 2025

Merged via the queue into mlflow:master with commit 1677244 Dec 17, 2025
46 checks passed

harupy deleted the deprecate-uc-functions branch December 17, 2025 10:42

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Deprecate Unity Catalog function integration in AI Gateway#19457

Deprecate Unity Catalog function integration in AI Gateway#19457
harupy merged 3 commits intomlflow:masterfrom
harupy:deprecate-uc-functions

harupy commented Dec 17, 2025 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

harupy commented Dec 17, 2025 •

edited by github-actions bot

Loading

Summary

Changes Made

Commit Details

Uh oh!

serena-ruan left a comment

Uh oh!

github-actions bot commented Dec 17, 2025 •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

harupy commented Dec 17, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Install mlflow from this PR

Related Issues/PRs

What changes are proposed in this pull request?

How is this PR tested?

Does this PR require documentation update?

Release Notes

Is this a user-facing change?

What component(s), interfaces, languages, and integrations does this PR affect?

How should the PR be classified in the release notes? Choose one:

Should this PR be included in the next patch release?

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

harupy commented Dec 17, 2025 • edited by github-actions bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Changes Made

Commit Details

Uh oh!

serena-ruan left a comment

Choose a reason for hiding this comment

Uh oh!

github-actions bot commented Dec 17, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

harupy commented Dec 17, 2025 •

edited

Loading

harupy commented Dec 17, 2025 •

edited by github-actions bot

Loading

github-actions bot commented Dec 17, 2025 •

edited

Loading