-
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
[CI] MultiMatchQueryBuilderTests#testToQuery fails #36598
Comments
Pinging @elastic/es-search |
That just reproducibly failed for me on 6.x with
So, I don't think it is fixed (at least in 6.x) |
I do find weird things when putting this test using
I have not yet determined if this there is state somewhere in test or the run order of tests, that may be an issue |
I think this is the offending code, which only exists in 6.x, but not in master https://github.com/elastic/elasticsearch/blob/6.x/server/src/main/java/org/elasticsearch/index/search/MultiMatchQuery.java#L277-L283 |
Due to the change in exception handling in elastic#36447 the backport now used different exceptions on parser failures between java and joda time formatting. This uses now IllegalArgumentExceptions for formatting errors all over the place. Closes elastic#36598
This fails reproducibly on current 6.x for me with
|
issue referencing the problem: #36598
Failing on 6.x with
|
only muting two failing test cases in a subclass where they fail. issue referencing the problem: elastic#36598
only muting two failing test cases in a subclass where they fail. issue referencing the problem: #36598
@spinscale Alex, I have created a fix for this test: #36742. The fix also workswith your PR #36604 |
This was addressed in #36742 |
Fails without reproduction in 6.x
https://elasticsearch-ci.elastic.co/job/elastic+elasticsearch+6.x+intake/602/console
The text was updated successfully, but these errors were encountered: