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.

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

[Meta] Lucene 9 Upgrade Punch List #2063

Closed
9 tasks done
nknize opened this issue Feb 7, 2022 · 9 comments · Fixed by #2531
Closed
9 tasks done

[Meta] Lucene 9 Upgrade Punch List #2063

nknize opened this issue Feb 7, 2022 · 9 comments · Fixed by #2531
Labels
>test-failure Test failure from CI, local build, etc. >upgrade Label used when upgrading library dependencies (e.g., Lucene) v2.0.0 Version 2.0.0

Comments

@nknize
Copy link
Collaborator

nknize commented Feb 7, 2022

The following outstanding (and currently muted) test failures still need to be addressed as part of the Lucene 9 upgrade:

  • PluginsServiceTests failures due to AccessControlException
  • CustomUnifiedHighlighterTests
  • HighlighterWithAnalyzersTests.testPhrasePrefix
  • ChildQuerySearchIT.testHighlightersIgnoreParentChild
  • HighlighterSearchIT failures
  • IndexShardHotspotTests
  • RelocationIT.testRelocationWhileIndexingRandom (seed -Dtests.seed=6158774967A60B81)
  • NumberFieldTypeTests.testIndexSortIntRange
  • IndexPrimaryRelocationIT ([Unmute] IndexPrimaryRelocationIT.testPrimaryRelocationWhileIndexing #2488)

relates #1109

@nknize nknize added >test-failure Test failure from CI, local build, etc. v2.0.0 Version 2.0.0 >upgrade Label used when upgrading library dependencies (e.g., Lucene) labels Feb 7, 2022
@reta
Copy link
Collaborator

reta commented Feb 18, 2022

These ones should be fixed now:

CustomUnifiedHighlighterTests
HighlighterWithAnalyzersTests.testPhrasePrefix
ChildQuerySearchIT.testHighlightersIgnoreParentChild
HighlighterSearchIT failures

But we would need to work on accommodating these change: https://github.com/apache/lucene/pull/362/files

@reta
Copy link
Collaborator

reta commented Mar 14, 2022

For NumberFieldTypeTests, opened https://issues.apache.org/jira/browse/LUCENE-10466

@reta
Copy link
Collaborator

reta commented Mar 16, 2022

PluginsServiceTests are green, #2484

@nknize
Copy link
Collaborator Author

nknize commented Mar 16, 2022

  • RelocationIT.testRelocationWhileIndexingRandom (seed -Dtests.seed=6158774967A60B81)

Fixed by #2452

@reta
Copy link
Collaborator

reta commented Mar 16, 2022

@nknize I am working on NumberFieldTypeTests.testIndexSortIntRange

@nknize
Copy link
Collaborator Author

nknize commented Mar 16, 2022

IndexShardHotspotTests are passing but this one test is taking upwards of 25 minutes+. @ashwinpankaj can we reduce this test from 100 indices + 5 shards to something much more expedient?

I don't want to unmute if this is going to kill our tests on each PR.

Also, a whole slew of the following logs:

...
[2022-03-16T12:08:09,916][INFO ][o.o.c.r.a.a.BalancedShardsAllocator] [testClusterScaleInWithSkew] Cannot move any shard in the cluster as there is no node on which shards can be allocated. Skipping shard iteration
[2022-03-16T12:08:11,283][INFO ][o.o.c.r.a.a.BalancedShardsAllocator] [testClusterScaleInWithSkew] Cannot move any shard in the cluster as there is no node on which shards can be allocated. Skipping shard iteration
[2022-03-16T12:08:12,343][INFO ][o.o.c.r.a.a.BalancedShardsAllocator] [testClusterScaleInWithSkew] Cannot move any shard in the cluster as there is no node on which shards can be allocated. Skipping shard iteration
[2022-03-16T12:08:13,229][INFO ][o.o.c.r.a.a.BalancedShardsAllocator] [testClusterScaleInWithSkew] Cannot move any shard in the cluster as there is no node on which shards can be allocated. Skipping shard iteration
[2022-03-16T12:08:13,846][INFO ][o.o.c.r.a.a.BalancedShardsAllocator] [testClusterScaleInWithSkew] Cannot move any shard in the cluster as there is no node on which shards can be allocated. Skipping shard iteration
[2022-03-16T12:08:16,243][INFO ][o.o.c.r.a.a.BalancedShardsAllocator] [testClusterScaleInWithSkew] Cannot move any shard in the cluster as there is no node on which shards can be allocated. Skipping shard iteration
[2022-03-16T12:08:19,200][INFO ][o.o.c.r.a.a.BalancedShardsAllocator] [testClusterScaleInWithSkew] Cannot move any shard in the cluster as there is no node on which shards can be allocated. Skipping shard iteration
[2022-03-16T12:08:20,625][INFO ][o.o.c.r.a.a.BalancedShardsAllocator] [testClusterScaleInWithSkew] Cannot move any shard in the cluster as there is no node on which shards can be allocated. Skipping shard iteration

@nknize
Copy link
Collaborator Author

nknize commented Mar 16, 2022

Documenting Points dimension failure from #2483 for posterity:

./gradlew ':server:internalClusterTest' --tests "org.opensearch.search.simple.SimpleSearchIT.testSimpleIndexSortEarlyTerminate" -Dtests.seed=E45EED588B09BBF2 -Dtests.security.manager=true -Dtests.jvm.argline="-XX:TieredStopAtLevel=1 -XX:ReservedCodeCacheSize=64m" -Dtests.locale=be-BY -Dtests.timezone=Pacific/Easter -Druntime.java=17
  1> java.lang.IllegalArgumentException: Field rank is indexed with 4 bytes per dimension, but org.apache.lucene.search.SortedNumericSortField$3@3f6c3860 expected 8
  1> 	at org.apache.lucene.search.comparators.NumericComparator$NumericLeafComparator.<init>(NumericComparator.java:116)
  1> 	at org.apache.lucene.search.comparators.LongComparator$LongLeafComparator.<init>(LongComparator.java:65)
  1> 	at org.apache.lucene.search.SortedNumericSortField$3$1.<init>(SortedNumericSortField.java:312)
  1> 	at org.apache.lucene.search.SortedNumericSortField$3.getLeafComparator(SortedNumericSortField.java:312)
  1> 	at org.apache.lucene.index.CheckIndex.testSort(CheckIndex.java:1109)
  1> 	at org.apache.lucene.index.CheckIndex.testSegment(CheckIndex.java:1012)
  1> 	at org.apache.lucene.index.CheckIndex.lambda$checkIndex$1(CheckIndex.java:762)
  1> 	at org.apache.lucene.index.CheckIndex.lambda$callableToSupplier$2(CheckIndex.java:860)
  1> 	at java.base/java.util.concurrent.CompletableFuture$AsyncSupply.run(CompletableFuture.java:1768)
  1> 	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136)
  1> 	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635)
  1> 	at java.base/java.lang.Thread.run(Thread.java:833)

@reta
Copy link
Collaborator

reta commented Mar 16, 2022

👍 @nknize that's the same cause as NumberFieldTypeTests.testIndexSortIntRange

@nknize
Copy link
Collaborator Author

nknize commented Mar 21, 2022

NumberFieldTypeTests are green; unmuting and closing in #2531. For any additional dimensional field type issues lets open a separate issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>test-failure Test failure from CI, local build, etc. >upgrade Label used when upgrading library dependencies (e.g., Lucene) v2.0.0 Version 2.0.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants