Skip to content
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

[CI] DissectParserTests.testBasicMatchUnicode #39244

Closed
albertzaharovits opened this issue Feb 21, 2019 · 2 comments · Fixed by #39262
Closed

[CI] DissectParserTests.testBasicMatchUnicode #39244

albertzaharovits opened this issue Feb 21, 2019 · 2 comments · Fixed by #39262
Assignees
@albertzaharovits
Labels
>test-failure Triaged test failures from CI

Comments

@albertzaharovits
Copy link
Contributor

albertzaharovits commented Feb 21, 2019 •
edited
Loading

https://elasticsearch-ci.elastic.co/job/elastic+elasticsearch+7.x+intake/240/console

This reproduces:

./gradlew :libs:dissect:unitTest \
  -Dtests.seed=EDD0A9CCDF94CE9 \
  -Dtests.class=org.elasticsearch.dissect.DissectParserTests \
  -Dtests.method="testBasicMatchUnicode" \
  -Dtests.security.manager=true \
  -Dtests.locale=be \
  -Dtests.timezone=Asia/Karachi \
  -Dcompiler.java=11 \
  -Druntime.java=8

java.lang.AssertionError: 
Expected: <[10Cc63zu0ymRmfY9Zk1, 3944OrI4JmcvtSZrSXxmMhVMqZDgNWvDezKE5NSI3Dx0BhVGeVMaSr1NF9zLtmlZiBujpCc3vgBhJiHrhddm2bbzb38O, 3z3ajPC8wqxakYDqCqq3tg894G5gKz8O, 4Cb7ehkVzJ2TPgsE0rgdZJU6PB1T, 4UDTbgxjn, BKy3sOon08jqSa3A9GvJJ, CH5an2fWSLfTG4OmAYl4z579WnzjjRViDtyK6TKemQvdXhXECCyLtHga1bFZviZJODPG5AiDEEviba9K, MhpF1MwHnZR0ImHGJ89FY9oGhHEd9uGCrEplvKWO0fidqGqqaX9HTJhh0czDWzYB89gOOgkjebOZ8pfdI84f7wLLbwH0kY8RO, RJ4K, Vgnp4mfwnLoGnD3UzOli27qb3y8JAVkY7CVWreu5r1RXE5KrSsP9DADb5pU478uTnv17Wg4PtaU1tbofEU, a08Be14bgaXvExDtLhduVn3Edo77pVtGtSOfe4KkxdDQovFYVBtEJjy0YXkd4Xwl78hfyRQbHSxr1mMSvwb9qKbyMsRo4, e, e, gCFLt2bweKr9zYkpqlwGRvnyjavnBZcI9YjxnhOE2kCWzRHnnqBQDu0yN2PFQHYyVaShvTNcD1T3o1NM3IKfUZpsng1kde, ghzUz, kdJ42wwuzr9Uqq5anJKKEq2rFTLziIBUjrfkjnUjaowFmE786SlY15fq15wr8kWsuA8AsbGL2vddTdDSKYksEvJeyAulUIN3, wMpYABhJ5VvTG2WOzP6gpIbw4DgagJtCY45imUGiOzIa1TLByAHd8B4jnKY0NEsoiJVYrrAkLYBB7BqgvqHlRQ37xMGu8bwOK8iZ, yVybK2, yboKhUQUsj1xaZQMCkRt1qr5PJyVjolC8eRVEUNsr2ud3CfpThmKQXPT]>
     but: was <[10Cc63zu0ymRmfY9Zk1, 3944OrI4JmcvtSZrSXxmMhVMqZDgNWvDezKE5NSI3Dx0BhVGeVMaSr1NF9zLtmlZiBujpCc3vgBhJiHrhddm2bbzb38O, 3z3ajPC8wqxakYDqCqq3tg894G5gKz8O, 4Cb7ehkVzJ2TPgsE0rgdZJU6PB1T, 4UDTbgxjn, BKy3sOon08jqSa3A9GvJJ, CH5an2fWSLfTG4OmAYl4z579WnzjjRViDtyK6TKemQvdXhXECCyLtHga1bFZviZJODPG5AiDEEviba9K, MhpF1MwHnZR0ImHGJ89FY9oGhHEd9uGCrEplvKWO0fidqGqqaX9HTJhh0czDWzYB89gOOgkjebOZ8pfdI84f7wLLbwH0kY8RO, RJ4K, Vgnp4mfwnLoGnD3UzOli27qb3y8JAVkY7CVWreu5r1RXE5KrSsP9DADb5pU478uTnv17Wg4PtaU1tbofEU, a08Be14bgaXvExDtLhduVn3Edo77pVtGtSOfe4KkxdDQovFYVBtEJjy0YXkd4Xwl78hfyRQbHSxr1mMSvwb9qKbyMsRo4, e, gCFLt2bweKr9zYkpqlwGRvnyjavnBZcI9YjxnhOE2kCWzRHnnqBQDu0yN2PFQHYyVaShvTNcD1T3o1NM3IKfUZpsng1kde, ghzUz, kdJ42wwuzr9Uqq5anJKKEq2rFTLziIBUjrfkjnUjaowFmE786SlY15fq15wr8kWsuA8AsbGL2vddTdDSKYksEvJeyAulUIN3, wMpYABhJ5VvTG2WOzP6gpIbw4DgagJtCY45imUGiOzIa1TLByAHd8B4jnKY0NEsoiJVYrrAkLYBB7BqgvqHlRQ37xMGu8bwOK8iZ, yVybK2, yboKhUQUsj1xaZQMCkRt1qr5PJyVjolC8eRVEUNsr2ud3CfpThmKQXPT]>
	at __randomizedtesting.SeedInfo.seed([EDD0A9CCDF94CE9:10BEBC69DFFCF02B]:0)
	at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:20)
	at org.junit.Assert.assertThat(Assert.java:956)
	at org.junit.Assert.assertThat(Assert.java:923)
	at org.elasticsearch.dissect.DissectParserTests.assertMatch(DissectParserTests.java:383)
	at org.elasticsearch.dissect.DissectParserTests.assertMatch(DissectParserTests.java:372)
	at org.elasticsearch.dissect.DissectParserTests.testBasicMatchUnicode(DissectParserTests.java:149)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at com.carrotsearch.randomizedtesting.RandomizedRunner.invoke(RandomizedRunner.java:1750)
	at com.carrotsearch.randomizedtesting.RandomizedRunner$8.evaluate(RandomizedRunner.java:938)
	at com.carrotsearch.randomizedtesting.RandomizedRunner$9.evaluate(RandomizedRunner.java:974)
	at com.carrotsearch.randomizedtesting.RandomizedRunner$10.evaluate(RandomizedRunner.java:988)
	at com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
	at org.apache.lucene.util.TestRuleSetupTeardownChained$1.evaluate(TestRuleSetupTeardownChained.java:49)
	at org.apache.lucene.util.AbstractBeforeAfterRule$1.evaluate(AbstractBeforeAfterRule.java:45)
	at org.apache.lucene.util.TestRuleThreadAndTestName$1.evaluate(TestRuleThreadAndTestName.java:48)
	at org.apache.lucene.util.TestRuleIgnoreAfterMaxFailures$1.evaluate(TestRuleIgnoreAfterMaxFailures.java:64)
	at org.apache.lucene.util.TestRuleMarkFailure$1.evaluate(TestRuleMarkFailure.java:47)
	at com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
	at com.carrotsearch.randomizedtesting.ThreadLeakControl$StatementRunner.run(ThreadLeakControl.java:368)
	at com.carrotsearch.randomizedtesting.ThreadLeakControl.forkTimeoutingTask(ThreadLeakControl.java:817)
	at com.carrotsearch.randomizedtesting.ThreadLeakControl$3.evaluate(ThreadLeakControl.java:468)
	at com.carrotsearch.randomizedtesting.RandomizedRunner.runSingleTest(RandomizedRunner.java:947)
	at com.carrotsearch.randomizedtesting.RandomizedRunner$5.evaluate(RandomizedRunner.java:832)
	at com.carrotsearch.randomizedtesting.RandomizedRunner$6.evaluate(RandomizedRunner.java:883)
	at com.carrotsearch.randomizedtesting.RandomizedRunner$7.evaluate(RandomizedRunner.java:894)
	at org.apache.lucene.util.AbstractBeforeAfterRule$1.evaluate(AbstractBeforeAfterRule.java:45)
	at com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
	at org.apache.lucene.util.TestRuleStoreClassName$1.evaluate(TestRuleStoreClassName.java:41)
	at com.carrotsearch.randomizedtesting.rules.NoShadowingOrOverridesOnMethodsRule$1.evaluate(NoShadowingOrOverridesOnMethodsRule.java:40)
	at com.carrotsearch.randomizedtesting.rules.NoShadowingOrOverridesOnMethodsRule$1.evaluate(NoShadowingOrOverridesOnMethodsRule.java:40)
	at com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
	at com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
	at com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
	at org.apache.lucene.util.TestRuleAssertionsRequired$1.evaluate(TestRuleAssertionsRequired.java:53)
	at org.apache.lucene.util.TestRuleMarkFailure$1.evaluate(TestRuleMarkFailure.java:47)
	at org.apache.lucene.util.TestRuleIgnoreAfterMaxFailures$1.evaluate(TestRuleIgnoreAfterMaxFailures.java:64)
	at org.apache.lucene.util.TestRuleIgnoreTestSuites$1.evaluate(TestRuleIgnoreTestSuites.java:54)
	at com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
	at com.carrotsearch.randomizedtesting.ThreadLeakControl$StatementRunner.run(ThreadLeakControl.java:368)
	at java.lang.Thread.run(Thread.java:748)

I'll be subtly snatching this from the ~ @elastic/es-search team~ @elastic/es-core-features , and try to fix it!
@albertzaharovits albertzaharovits added :Search Relevance/Analysis How text is split into tokens >test-failure Triaged test failures from CI labels Feb 21, 2019
@albertzaharovits albertzaharovits self-assigned this Feb 21, 2019
@albertzaharovits
Copy link
Contributor Author

And mute, ofc!

@albertzaharovits albertzaharovits mentioned this issue Feb 21, 2019
Merged
@albertzaharovits albertzaharovits added :Core/Features/Features and removed :Search Relevance/Analysis How text is split into tokens labels Feb 21, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-features

@albertzaharovits albertzaharovits mentioned this issue Feb 21, 2019
Merged
albertzaharovits added a commit that referenced this issue Feb 22, 2019
@albertzaharovits
Fix DissectParserTests expecting unique keys (#39262)
1e7f28e 
Fixes a bug in DissectParserTests where the tests expected dissect
keys to be unique but were not.

Closes #39244
albertzaharovits added a commit that referenced this issue Feb 22, 2019
@albertzaharovits
Fix DissectParserTests expecting unique keys (#39262)
045c9c4 
Fixes a bug in DissectParserTests where the tests expected dissect
keys to be unique but were not.

Closes #39244
albertzaharovits added a commit that referenced this issue Feb 22, 2019
@albertzaharovits
Fix DissectParserTests expecting unique keys (#39262)
be14c60 
Fixes a bug in DissectParserTests where the tests expected dissect
keys to be unique but were not.

Closes #39244
albertzaharovits added a commit that referenced this issue Feb 22, 2019
@albertzaharovits
Fix DissectParserTests expecting unique keys (#39262)
ca630bb 
Fixes a bug in DissectParserTests where the tests expected dissect
keys to be unique but were not.

Closes #39244
weizijun pushed a commit to weizijun/elasticsearch that referenced this issue Feb 22, 2019
@albertzaharovits @weizijun
Fix DissectParserTests expecting unique keys (elastic#39262)
53dcdc3 
Fixes a bug in DissectParserTests where the tests expected dissect
keys to be unique but were not.

Closes elastic#39244
weizijun pushed a commit to weizijun/elasticsearch that referenced this issue Feb 22, 2019
@albertzaharovits @weizijun
Fix DissectParserTests expecting unique keys (elastic#39262)
4eada3e 
Fixes a bug in DissectParserTests where the tests expected dissect
keys to be unique but were not.

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

@albertzaharovits albertzaharovits

Labels
>test-failure Triaged test failures from CI
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix DissectParserTests expecting unique keys albertzaharovits/elasticsearch
2 participants
@albertzaharovits @elasticmachine