Skip to content

Add forceMerge to TestIndexOptions#testFrozenFieldTypeCacheRestabilizes#15946

Merged
romseygeek merged 1 commit into
apache:mainfrom
romseygeek:test/frozen-fieldinfo-forcemerge
Apr 9, 2026
Merged

Add forceMerge to TestIndexOptions#testFrozenFieldTypeCacheRestabilizes#15946
romseygeek merged 1 commit into
apache:mainfrom
romseygeek:test/frozen-fieldinfo-forcemerge

Conversation

@romseygeek
Copy link
Copy Markdown
Contributor

The test assumes we have a single segment, but randomization can sometimes give us more than that. Adding a forceMerge call ensures the segment geometry is what we expect it to be.

Fixes #15943

@romseygeek romseygeek self-assigned this Apr 9, 2026
@romseygeek romseygeek added the skip-changelog Apply to PRs that don't need a changelog entry, stopping the automated changelog check. label Apr 9, 2026
The test assumes we have a single segment, but randomization can
sometimes give us more than that.  Adding a forceMerge call ensures
the segment geometry is what we expect it to be.

Fixes apache#15943
@romseygeek romseygeek force-pushed the test/frozen-fieldinfo-forcemerge branch from 436b974 to 8ca3be7 Compare April 9, 2026 10:13
@romseygeek romseygeek merged commit 7ca782c into apache:main Apr 9, 2026
13 checks passed
@romseygeek romseygeek deleted the test/frozen-fieldinfo-forcemerge branch April 9, 2026 11:17
romseygeek added a commit that referenced this pull request Apr 9, 2026
…es (#15946)

The test assumes we have a single segment, but randomization can
sometimes give us more than that.  Adding a forceMerge call ensures
the segment geometry is what we expect it to be.

Fixes #15943
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module:core/index skip-changelog Apply to PRs that don't need a changelog entry, stopping the automated changelog check.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Failing test in TestIndexOptions: testFrozenFieldTypeCacheRestabilizes

1 participant