Skip to content

Remove hppc from index translog#85657

Merged
rjernst merged 3 commits intoelastic:masterfrom
OlivierCavadenti:remove-hppc-translog
Apr 26, 2022
Merged

Remove hppc from index translog#85657
rjernst merged 3 commits intoelastic:masterfrom
OlivierCavadenti:remove-hppc-translog

Conversation

@OlivierCavadenti
Copy link
Copy Markdown
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/Engine Anything around managing Lucene and the Translog in an open shard. >refactoring labels Apr 21, 2022
@elasticmachine elasticmachine added the Team:Distributed Meta label for distributed team. label Apr 21, 2022
@elasticmachine
Copy link
Copy Markdown
Collaborator

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

@rjernst
Copy link
Copy Markdown
Member

rjernst commented Apr 21, 2022

@elasticmachine ok to test

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

rjernst commented Apr 21, 2022

@elasticmachine update branch

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Contributor

@elasticmachine update branch

Copy link
Copy Markdown
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed/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 Meta label for distributed team. v8.3.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants