Skip to content

Comments

tree aggregation test#90

Open
alexgolod wants to merge 2 commits intomainfrom
fix_tree_aggregation
Open

tree aggregation test#90
alexgolod wants to merge 2 commits intomainfrom
fix_tree_aggregation

Conversation

@alexgolod
Copy link
Contributor

No description provided.

@alexgolod alexgolod requested review from Copilot and pablopnn February 5, 2026 15:15
@alexgolod alexgolod self-assigned this Feb 5, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request adds a new test function to verify the ownership preference logic during tree aggregation in the pool controller helpers. The test ensures that cluster data from owning controllers is correctly prioritized over data from non-owning controllers.

Changes:

  • Added TestTreeAggregationOwnershipPreference test function with comprehensive test cases covering various ownership scenarios

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants