Skip to content

Remove hppc from index translog #85657

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.

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

Merged
merged 3 commits into from
Apr 26, 2022

Conversation

OlivierCavadenti
Copy link
Contributor

Remove LongObjectHashMap, LongArrayList and LongProcedure from index translog files.

relates #84735

Remove LongObjectHashMap, LongArrayList and LongProcedure from index translog files.

relates elastic#84735
@elasticsearchmachine elasticsearchmachine added external-contributor Pull request authored by a developer outside the Elasticsearch team v8.3.0 labels Apr 3, 2022
@jtibshirani jtibshirani added :Distributed Indexing/Engine Anything around managing Lucene and the Translog in an open shard. >refactoring labels Apr 21, 2022
@elasticmachine elasticmachine added the Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. label Apr 21, 2022
@elasticmachine
Copy link
Collaborator

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

@rjernst
Copy link
Member

rjernst commented Apr 21, 2022

@elasticmachine ok to test

@rjernst rjernst mentioned this pull request Apr 21, 2022
43 tasks
@rjernst
Copy link
Member

rjernst commented Apr 21, 2022

@elasticmachine update branch

Copy link
Contributor

@original-brownbear original-brownbear left a comment

Choose a reason for hiding this comment

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

Sorry taking this back

Copy link
Contributor

@original-brownbear original-brownbear left a comment

Choose a reason for hiding this comment

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

Sorry removing previous review, gotta look into this some more.

@original-brownbear original-brownbear dismissed their stale review April 24, 2022 20:02

misread something

@original-brownbear original-brownbear self-requested a review April 24, 2022 20:02
@original-brownbear
Copy link
Contributor

@elasticmachine update branch

Copy link
Member

@rjernst rjernst left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks @OlivierCavadenti

@rjernst rjernst merged commit 17d877f into elastic:master Apr 26, 2022
original-brownbear added a commit to original-brownbear/elasticsearch that referenced this pull request May 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Distributed Indexing/Engine Anything around managing Lucene and the Translog in an open shard. external-contributor Pull request authored by a developer outside the Elasticsearch team >refactoring Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. v8.3.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants