Skip to content

CI: SimpleQueryStringIT.testDocWithAllTypes fails periodically #44192

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
Closed
imotov opened this issue Jul 10, 2019 · 2 comments · Fixed by #44215
Closed

CI: SimpleQueryStringIT.testDocWithAllTypes fails periodically #44192

imotov opened this issue Jul 10, 2019 · 2 comments · Fixed by #44215
Assignees
@cbuescher
Labels
:Search/Search Search-related issues that do not fall into other categories >test-failure Triaged test failures from CI

Comments

@imotov
Copy link
Contributor

imotov commented Jul 10, 2019

Introduction of https://issues.apache.org/jira/browse/LUCENE-8811 that was enabled by #44171 makes this test to fail with TooManyClauses[maxClauseCount is set to 52]; error since the last query generates 57 clauses.

REPRODUCE WITH: ./gradlew :server:integTest --tests "org.elasticsearch.search.query.SimpleQueryStringIT.testDocWithAllTypes" -Dtests.seed=4BE99E7769CF3EAC -Dtests.security.manager=true -Dtests.locale=dav-KE -Dtests.timezone=US/Pacific -Dcompiler.java=12 -Druntime.java=11
@imotov imotov added :Search/Search Search-related issues that do not fall into other categories >test-failure Triaged test failures from CI labels Jul 10, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-search

imotov added a commit that referenced this issue Jul 10, 2019
@imotov
CI: Disable SimpleQueryStringIT.testDocWithAllTypes
3702bc3 
Tracked by #44192
@cbuescher cbuescher self-assigned this Jul 11, 2019
@cbuescher
Copy link
Member

For some reason we randomize the CLUSTER_MAX_CLAUSE_COUNT on test setup (@BeforeClass) between 50 and 100. For the query in question we seem to need at least 57 clauses to pass. This probably hasn't been detected before. I don't see a reason we need to be that conservative in the allowed number of clauses and will raise it.

cbuescher pushed a commit to cbuescher/elasticsearch that referenced this issue Jul 11, 2019
[Tests] Increase SimpleQueryStringIT allowed maxClauseCount
aac8670 
For this test, we randomize the CLUSTER_MAX_CLAUSE_COUNT on test setup
(@BeforeClass) between 50 and 100. Some queries in the test generate 56 clauses
which hasn't been an issue before LUCENE-8811, but we slightly need to increase
the minimal possible clause count now.

Closes elastic#44192
@cbuescher cbuescher mentioned this issue Jul 11, 2019
Merged
nknize pushed a commit that referenced this issue Jul 11, 2019
@imotov @nknize
CI: Disable SimpleQueryStringIT.testDocWithAllTypes
1636701 
Tracked by #44192
cbuescher pushed a commit that referenced this issue Jul 11, 2019
[Tests] Increase SimpleQueryStringIT allowed maxClauseCount (#44215)
31725ef 
For this test, we randomize the CLUSTER_MAX_CLAUSE_COUNT on test setup
(@BeforeClass) between 50 and 100. Some queries in the test generate 56 clauses
which hasn't been an issue before LUCENE-8811, but we slightly need to increase
the minimal possible clause count now.

Closes #44192
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cbuescher cbuescher

Labels
:Search/Search Search-related issues that do not fall into other categories >test-failure Triaged test failures from CI
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Tests] Increase SimpleQueryStringIT allowed maxClauseCount cbuescher/elasticsearch
3 participants
@imotov @cbuescher @elasticmachine