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.

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

Make RoutingNodes behave like a collection #83540

Merged
merged 1 commit into from
Feb 7, 2022

Conversation

arteam
Copy link
Contributor

@arteam arteam commented Feb 4, 2022

Extend it from AbstractCollection instead of Iterable analogous to #83453

@arteam arteam added the :Distributed/Distributed A catch all label for anything in the Distributed Area. If you aren't sure, use this one. label Feb 4, 2022
@arteam arteam force-pushed the routing-node-abstract-collection branch from d961440 to 7cee850 Compare February 4, 2022 17:18
Extend it from `AbstractCollection` instead of `Iterable` analogous to elastic#83453
@arteam arteam force-pushed the routing-node-abstract-collection branch from 7cee850 to 78913c5 Compare February 4, 2022 17:29
@arteam arteam marked this pull request as ready for review February 4, 2022 18:54
@elasticmachine elasticmachine added the Team:Distributed Meta label for distributed team label Feb 4, 2022
@elasticmachine
Copy link
Collaborator

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

@arteam arteam added auto-backport Automatically create backport pull requests when merged v8.1.1 labels Feb 7, 2022
@arteam arteam merged commit a12de4c into elastic:master Feb 7, 2022
@arteam arteam deleted the routing-node-abstract-collection branch February 7, 2022 08:24
arteam added a commit to arteam/elasticsearch that referenced this pull request Feb 7, 2022
Extend it from `AbstractCollection` instead of `Iterable` analogous to elastic#83453
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
8.1

@arteam
Copy link
Contributor Author

arteam commented Feb 7, 2022

Thanks Ievgen!

arteam added a commit that referenced this pull request Feb 7, 2022
Extend it from `AbstractCollection` instead of `Iterable` analogous to #83453
weizijun added a commit to weizijun/elasticsearch that referenced this pull request Feb 7, 2022
* upstream/master:
  [DOCS] Switch xrefs to external links (elastic#83590)
  [DOCS] 'features' flag added in elastic#83083 (elastic#83452)
  Rename ChangePolicyforIndexIT to ChangePolicyForIndexIT (elastic#83569)
  Fixing random_sampler tests (elastic#83549)
  Upgrade Checkstyle to 9.3 (elastic#83314)
  Make improvements to the release notes generator (elastic#83525)
  Cleanup DataTierAllocationDecider (elastic#83572)
  Upgrade jANSI dependency to 2.4.0 (elastic#83566)
  Speed up Name Collision Check in Metadata.Builder (elastic#83340)
  SQL: Add range checks to interval multiplication operation (elastic#83478)
  Remove DiscoveryNodes#getAllNodes (elastic#83538)
  Make RoutingNodes behave like a collection (elastic#83540)
  Remove Unused CS Listener from SecurityServerTransportInterceptor (elastic#83556)
arteam added a commit that referenced this pull request Feb 14, 2022
arteam added a commit that referenced this pull request Feb 14, 2022
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 :Distributed/Distributed A catch all label for anything in the Distributed Area. If you aren't sure, use this one. >non-issue >refactoring Team:Distributed Meta label for distributed team v8.1.1 v8.2.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants