Skip to content

Commit dd9f099

Browse files
patelchaitanyntkathole
authored andcommitted
fix: Remove runtime pip install hints from mlflow errors
A disconnected-readiness scan (python-imports-bundled) flags these messages because they instruct users to run pip at runtime, which fails in air-gapped environments that have no internet access or internal mirror. Point at the 'feast[mlflow]' extra instead of a pip command. Signed-off-by: Chaitany Patel <[email protected]>
1 parent 9c2da18 commit dd9f099

3 files changed

Lines changed: 3 additions & 3 deletions

File tree

‎sdk/python/feast/cli/mlflow_cmd.py‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ def _require_mlflow() -> None:
1818
import mlflow # noqa: F401
1919
except ImportError as e:
2020
raise click.ClickException(
21-
"The 'mlflow' package is required. Install it with: pip install 'feast[mlflow]'"
21+
"The 'mlflow' package is required. Install the 'feast[mlflow]' extra in your environment."
2222
) from e
2323

2424

‎sdk/python/feast/mlflow.py‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ def _build_client() -> Any:
8080
except ImportError:
8181
raise ImportError(
8282
"mlflow package is not installed. "
83-
"Install it with: pip install feast[mlflow]"
83+
"Install the 'feast[mlflow]' extra in your environment."
8484
)
8585

8686

‎sdk/python/feast/mlflow_integration/dataset_sync.py‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,7 @@ def sync_mlflow_dataset_to_feast(
8181
except ImportError as e:
8282
raise ImportError(
8383
"The 'mlflow' package is required for dataset sync. "
84-
"Install it with: pip install 'feast[mlflow]'"
84+
"Install the 'feast[mlflow]' extra in your environment."
8585
) from e
8686

8787
result = SyncResult()

0 commit comments

Comments
 (0)