Skip to content

Mark MlFlushJobRequest as deprecated with future version notice #3695

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.

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

Closed
wants to merge 4 commits into from

Conversation

valeriy42
Copy link
Contributor

Mark MlFlushJobRequest as deprecated with future version notice

@valeriy42 valeriy42 requested review from a team as code owners February 4, 2025 14:58
Copy link
Contributor Author

@valeriy42 valeriy42 left a comment

Choose a reason for hiding this comment

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

This should be merged after elastic/elasticsearch#121667 is merged.

@valeriy42 valeriy42 self-assigned this Feb 4, 2025
@valeriy42 valeriy42 requested review from szabosteve and removed request for a team February 4, 2025 14:59
@valeriy42 valeriy42 added on hold ml skip-backport This pull request should not be backported labels Feb 4, 2025
Copy link
Contributor

@szabosteve szabosteve left a comment

Choose a reason for hiding this comment

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

LGTM

@valeriy42
Copy link
Contributor Author

I replaced this PR with #3726 so the CI validation can work properly.

valeriy42 added a commit that referenced this pull request Feb 6, 2025
Mark MlFlushJobRequest as deprecated with a future version notice.

Superseeds #3695 to comply with the validation workflows.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ml on hold skip-backport This pull request should not be backported specification
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants