Skip to content

Conversation

andreidan
Copy link
Contributor

This fixes testRetryPointInTime which on teardown is looking to assert that the operations in the translog and in the lucene index are the same.

Previously we didn't wait for the translog operations to be applied. This changes assertConsistentHistoryInLuceneIndex to wait for the pending operations in the translog to be applied.

Fixes #117116

(cherry picked from commit b4e852a)
Signed-off-by: Andrei Dan andrei.dan@elastic.co

Backport of #118244

) This fixes testRetryPointInTime which on teardown is looking to assert that the operations in the translog and in the lucene index are the same. Previously we didn't wait for the translog operations to be applied. This changes `assertConsistentHistoryInLuceneIndex` to wait for the pending operations in the translog to be applied. Fixes elastic#117116 (cherry picked from commit b4e852a) Signed-off-by: Andrei Dan <andrei.dan@elastic.co>
@andreidan andreidan added >test Issues or PRs that are addressing/adding tests backport :Search Foundations/Search Catch all for Search Foundations auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) labels Dec 9, 2024
@elasticsearchmachine elasticsearchmachine merged commit 1e62d47 into elastic:8.x Dec 9, 2024
15 checks passed
@andreidan andreidan deleted the backport/8.x-fix-fix-testRetryPointInTime branch December 9, 2024 13:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport :Search Foundations/Search Catch all for Search Foundations >test Issues or PRs that are addressing/adding tests v8.18.0

2 participants