-
Notifications
You must be signed in to change notification settings - Fork 25.2k
Verify that the failure reason of analytics process is empty #48042
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
Verify that the failure reason of analytics process is empty #48042
Conversation
Pinging @elastic/ml-core (:ml) |
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, although I left one suggestion about how we might avoid frustration in the future.
...est/java/org/elasticsearch/xpack/ml/integration/MlNativeDataFrameAnalyticsIntegTestCase.java
Outdated
Show resolved
Hide resolved
run elasticsearch-ci/bwc |
5 similar comments
run elasticsearch-ci/bwc |
run elasticsearch-ci/bwc |
run elasticsearch-ci/bwc |
run elasticsearch-ci/bwc |
run elasticsearch-ci/bwc |
fc6a4c8
to
af65ba8
Compare
run elasticsearch-ci/packaging-sample-matrix |
run elasticsearch-ci/2 |
To help debugging issue #47945 we need more information printed out when the test fails.
This PR exposes
stats.failureReason
(which we already have) in the test output.Relates #47945