Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Pass checkpoint to commit extra user data function #122930

Merged
merged 1 commit into from
Feb 19, 2025
Merged

Pass checkpoint to commit extra user data function #122930

merged 1 commit into from
Feb 19, 2025

Conversation

kingherc
Copy link
Contributor

Relates ES-10852

@kingherc kingherc added >non-issue :Distributed Indexing/Engine Anything around managing Lucene and the Translog in an open shard. Team:Distributed Indexing Meta label for Distributed Indexing team v9.1.0 labels Feb 19, 2025
@kingherc kingherc self-assigned this Feb 19, 2025
@elasticsearchmachine elasticsearchmachine added the serverless-linked Added by automation, don't add manually label Feb 19, 2025
@elasticsearchmachine
Copy link
Collaborator

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

tlrx
tlrx approved these changes Feb 19, 2025
View reviewed changes
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

fcofdez
fcofdez approved these changes Feb 19, 2025
View reviewed changes
Copy link
Contributor

@fcofdez fcofdez left a comment

Choose a reason for hiding this comment

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

LGTM

@kingherc kingherc merged commit ad220c1 into elastic:main Feb 19, 2025
18 checks passed
@kingherc kingherc deleted the non-issue/ES-10852-fix-conc-hollow-flush branch February 19, 2025 14:14
arteam
arteam reviewed Feb 19, 2025
View reviewed changes
Copy link
Contributor

@arteam arteam left a comment

Choose a reason for hiding this comment

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

LGTM3

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

@arteam arteam arteam left review comments

@tlrx tlrx tlrx approved these changes

@fcofdez fcofdez fcofdez approved these changes

@henningandersen henningandersen Awaiting requested review from henningandersen

Assignees

@kingherc kingherc

Labels
:Distributed Indexing/Engine Anything around managing Lucene and the Translog in an open shard. >non-issue serverless-linked Added by automation, don't add manually Team:Distributed Indexing Meta label for Distributed Indexing team v9.1.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@kingherc @elasticsearchmachine @tlrx @fcofdez @arteam