Skip to content

[Test] Fix SparseFileTrackerTests.testCallsListenerWhenRangeIsAvailable failing#109239

Merged
tlrx merged 1 commit intoelastic:mainfrom
tlrx:2024/05/31/fix-109237
May 31, 2024
Merged

[Test] Fix SparseFileTrackerTests.testCallsListenerWhenRangeIsAvailable failing#109239
tlrx merged 1 commit intoelastic:mainfrom
tlrx:2024/05/31/fix-109237

Conversation

@tlrx
Copy link
Copy Markdown
Member

@tlrx tlrx commented May 31, 2024

Test required some adjustments now listeners are
not completed on progress update if they are waiting up to the end of the gap.

Relates #108095
Closes #109237

…le failing

Test required some adjustments now listeners are
not completed on progress update if they are waiting
up to the end of the gap.

Relates elastic#108095
Closes elastic#109237
@tlrx tlrx added >test Issues or PRs that are addressing/adding tests :Distributed/Distributed A catch all label for anything in the Distributed Area. Please avoid if you can. v8.15.0 labels May 31, 2024
@tlrx tlrx requested review from henningandersen and ywangd May 31, 2024 10:24
@elasticsearchmachine elasticsearchmachine added the Team:Distributed Meta label for distributed team. label May 31, 2024
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

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

Copy link
Copy Markdown
Contributor

@henningandersen henningandersen left a comment

Choose a reason for hiding this comment

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

LGTM

@tlrx tlrx merged commit 42f1e1d into elastic:main May 31, 2024
@tlrx tlrx deleted the 2024/05/31/fix-109237 branch May 31, 2024 11:06
@tlrx
Copy link
Copy Markdown
Member Author

tlrx commented May 31, 2024

Thanks Henning! And sorry for the noise.

ywangd added a commit to ywangd/elasticsearch that referenced this pull request Jun 3, 2024
ywangd added a commit that referenced this pull request Jun 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed/Distributed A catch all label for anything in the Distributed Area. Please avoid if you can. Team:Distributed Meta label for distributed team. >test Issues or PRs that are addressing/adding tests v8.15.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] SparseFileTrackerTests testCallsListenerWhenRangeIsAvailable failing

3 participants