-
Notifications
You must be signed in to change notification settings - Fork 24.7k
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
Remove ChunkedToXContentHelper.array method, swap for Builder #114319
Conversation
@elasticmachine update branch |
@elasticmachine update branch |
Pinging @elastic/es-core-infra (Team:Core/Infra) |
3491e5a
to
355680e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, but I'm not sure I would change the Iterator in the *Decision.java classes, it is one of those cases where it looks more natural.
...r/src/main/java/org/elasticsearch/cluster/routing/allocation/AbstractAllocationDecision.java
Outdated
Show resolved
Hide resolved
💔 Backport failedThe backport operation could not be completed due to the following error:
You can use sqren/backport to manually backport by running |
No description provided.