Skip to content

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