Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add better message for file store uris. #114

Merged
merged 1 commit into from
Jan 22, 2025
Merged

Add better message for file store uris. #114

merged 1 commit into from
Jan 22, 2025

Conversation

nojaf
Copy link
Collaborator

@nojaf nojaf commented Jan 22, 2025

When running mlflow server without any further arguments, mlflow will use the file store by default and not sqlite.

I want to make it more clear for users trying mlflow-go server without any argument.

@nojaf
Add better message for file store uris.
ed7dded 
Signed-off-by: nojaf <florian.verdonck@outlook.com>
@nojaf nojaf requested a review from dsuhinin January 22, 2025 11:01
dsuhinin
dsuhinin approved these changes Jan 22, 2025
View reviewed changes
Copy link
Collaborator

@dsuhinin dsuhinin left a comment

Choose a reason for hiding this comment

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

Looks good!

@nojaf nojaf merged commit ed0105f into mlflow:main Jan 22, 2025
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dsuhinin dsuhinin dsuhinin approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@nojaf @dsuhinin