Skip to content

Conversation

@javanna
Copy link
Member

@javanna javanna commented Feb 6, 2025

ShardIterator is an interface with a single implementation called PlainShardIterator. This commit makes it a concrete final class and folds its only implementation into it.

ShardIterator is an interface with a single implementation called PlainShardIterator.
This commit makes it a concrete final class and folds its only implementation into it.
@javanna javanna added :Distributed Indexing/Distributed A catch all label for anything in the Distributed Indexing Area. Please avoid if you can. >refactoring v8.19.0 v9.1.0 labels Feb 6, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@elasticsearchmachine elasticsearchmachine added the Team:Distributed Indexing Meta label for Distributed Indexing team label Feb 6, 2025
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.

LGTM

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, left some comments for the javadoc

@javanna javanna added the auto-backport Automatically create backport pull requests when merged label Feb 6, 2025
@javanna javanna merged commit f2b0905 into elastic:main Feb 6, 2025
17 checks passed
@javanna javanna deleted the refactoring/remove_plain_shard_iterator branch February 6, 2025 16:29
@javanna
Copy link
Member Author

javanna commented Feb 6, 2025

Thanks, both of you Armin and Tanguy!

@elasticsearchmachine
Copy link
Collaborator

💔 Backport failed

Status Branch Result
8.x Commit could not be cherrypicked due to conflicts

You can use sqren/backport to manually backport by running backport --upstream elastic/elasticsearch --pr 121893

elasticsearchmachine pushed a commit that referenced this pull request Feb 12, 2025
ShardIterator is an interface with a single implementation called PlainShardIterator.
This commit makes it a concrete final class and folds its only implementation into it.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Automatically create backport pull requests when merged backport pending :Distributed Indexing/Distributed A catch all label for anything in the Distributed Indexing Area. Please avoid if you can. >refactoring Team:Distributed Indexing Meta label for Distributed Indexing team v8.19.0 v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants