Skip to content

Revisit loggers and log levels for desired balance allocator#90440

Merged
idegtiarenko merged 5 commits intoelastic:feature/desired-balance-allocatorfrom
idegtiarenko:revisit_log_levels
Sep 30, 2022
Merged

Revisit loggers and log levels for desired balance allocator#90440
idegtiarenko merged 5 commits intoelastic:feature/desired-balance-allocatorfrom
idegtiarenko:revisit_log_levels

Conversation

@idegtiarenko
Copy link
Copy Markdown
Contributor

No description provided.

@idegtiarenko idegtiarenko added :Distributed/Allocation All issues relating to the decision making around placing a shard (both master logic & on the nodes) Team:Distributed Meta label for distributed team. labels Sep 28, 2022
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

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

@idegtiarenko
Copy link
Copy Markdown
Contributor Author

This should be merged after #89900 as that changes DesiredBalanceShardsAllocator that might require additional logging changes.

# Conflicts:
#	server/src/main/java/org/elasticsearch/cluster/routing/allocation/allocator/DesiredBalanceShardsAllocator.java
Copy link
Copy Markdown
Member

@DaveCTurner DaveCTurner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@idegtiarenko idegtiarenko merged commit 956a404 into elastic:feature/desired-balance-allocator Sep 30, 2022
@idegtiarenko idegtiarenko deleted the revisit_log_levels branch September 30, 2022 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed/Allocation All issues relating to the decision making around placing a shard (both master logic & on the nodes) Team:Distributed Meta label for distributed team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants