Skip to content

Conversation

@dnhatn
Copy link
Member

@dnhatn dnhatn commented Mar 18, 2022

Backports the following commits to 8.1:

Snapshotted commits should also hold a reference to the store, so they 
are always usable; otherwise, callers need to manage the store's 
references manually. This change applies only to InternalEngine as we
already do this in ReadOnlyEngine.
@dnhatn dnhatn added auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport labels Mar 18, 2022
@elasticsearchmachine elasticsearchmachine merged commit 298d625 into elastic:8.1 Mar 18, 2022
@dnhatn dnhatn deleted the backport/8.1/pr-84776 branch March 18, 2022 16:08
dnhatn added a commit to dnhatn/elasticsearch that referenced this pull request Mar 26, 2022
…lastic#85112)

Snapshotted commits should also hold a reference to the store, so they
are always usable; otherwise, callers need to manage the store's
references manually. This change applies only to InternalEngine as we
already do this in ReadOnlyEngine.
elasticsearchmachine pushed a commit that referenced this pull request Mar 26, 2022
…#85365)

Snapshotted commits should also hold a reference to the store, so they
are always usable; otherwise, callers need to manage the store's
references manually. This change applies only to InternalEngine as we
already do this in ReadOnlyEngine.
@dnhatn dnhatn removed auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) v8.1.2 labels Mar 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants