Skip to content

Conversation

original-brownbear
Copy link
Contributor

Create more segments here to make sure the background merge always merges.
With just 3 segments and a max-segments-per-tier of 2 we don't have the guarantee
that a merge will actually run and hence the test will fail when waiting for the background
merge.

closes #89412

…roundMerges

Create more segments here to make sure the background merge always merges.
With just 3 segments and a max-segments-per-tier of 2 we don't have the guarantee
that a merge will actually run and hence the test will fail when waiting for the background
merge.

closes elastic#89412
@original-brownbear original-brownbear added >test Issues or PRs that are addressing/adding tests :Distributed Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs v8.4.1 v8.5.0 labels Aug 17, 2022
@elasticsearchmachine elasticsearchmachine added the Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. label Aug 17, 2022
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-distributed (Team:Distributed)

Copy link
Member

@tlrx tlrx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@original-brownbear
Copy link
Contributor Author

Thanks Tanguy!

@original-brownbear original-brownbear merged commit 2a08258 into elastic:main Aug 17, 2022
@original-brownbear original-brownbear deleted the 89412 branch August 17, 2022 13:13
original-brownbear added a commit to original-brownbear/elasticsearch that referenced this pull request Aug 17, 2022
…roundMerges (elastic#89416)

Create more segments here to make sure the background merge always merges.
With just 3 segments and a max-segments-per-tier of 2 we don't have the guarantee
that a merge will actually run and hence the test will fail when waiting for the background
merge.

closes elastic#89412
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
8.4

elasticsearchmachine pushed a commit that referenced this pull request Aug 17, 2022
…roundMerges (#89416) (#89423)

Create more segments here to make sure the background merge always merges.
With just 3 segments and a max-segments-per-tier of 2 we don't have the guarantee
that a merge will actually run and hence the test will fail when waiting for the background
merge.

closes #89412
@mark-vieira mark-vieira added v8.4.0 and removed v8.4.1 labels Aug 24, 2022
@original-brownbear original-brownbear restored the 89412 branch November 30, 2024 10:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Distributed Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. >test Issues or PRs that are addressing/adding tests v8.4.0 v8.5.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

BlobStoreIncrementalityIT.testRecordCorrectSegmentCountsWithBackgroundMerges
4 participants