dlmarion commented on code in PR #5530:
URL: https://github.com/apache/accumulo/pull/5530#discussion_r2102684523
##########
server/manager/src/main/java/org/apache/accumulo/manager/Manager.java:
##########
@@ -1918,10 +1918,20 @@ public boolean isUpgrading() {
}
void initializeBalancer() {
- var localTabletBalancer =
Property.createInstanceFromPropertyName(getConfiguration(),
- Property.MANAGER_TABLET_BALANCER, TabletBalancer.class, new
TableLoadBalancer());
- localTabletBalancer.init(balancerEnvironment);
- tabletBalancer = localTabletBalancer;
+ try {
+ getContext().getPropStore().getCache().removeAll();
Review Comment:
`PropStoreWatcher.process` calls `PropStoreWatcher.signalZkChangeEvent` when
the Watcher is fired for the property change. `signalZkChangeEvent` puts a
Runnable on a queue for an ExecutorService, which when fired ends up marking
the PropStore as needing an update. You should be able to enable `trace`
logging for `PropStoreWatcher` to see when the Watcher receives the
notification. You should also be able to see when `PropSnapshot.zkChangeEvent`
is called as that is logged at debug.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscr...@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
