-
Notifications
You must be signed in to change notification settings - Fork 24.9k
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
[ML] Fixing duplicate audit messages in delayed data check #37080
[ML] Fixing duplicate audit messages in delayed data check #37080
Conversation
Pinging @elastic/ml-core |
Since there are other blockers open against 6.6.0 I think the complete fix should be backported into 6.6.0. It would be best if this new functionality didn’t start off with a reputation for spamming users. Also, please change the PR title to something that makes sense as a standalone sentence for the release notes. |
run the gradle build tests 1 |
1 similar comment
run the gradle build tests 1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Fixes the bug where multiple annotations will occur if there is an end time that is different than the timestamp.
Minor other changes as well for consistency with the UI.
closes #36891 #36888 #36889