Skip to content

Commit

Permalink
Fix updated config not notifying config registry
Browse files Browse the repository at this point in the history
Add update notify to master

This issue introduced by #13514

pr-link: #16764
change-id: cid-36f2010233c3af95f28cf9cb7ac0a0a1963c86fe
  • Loading branch information
maobaolong committed Feb 3, 2023
1 parent 6ca7d15 commit 407cf15
Showing 1 changed file with 4 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@
import alluxio.conf.Configuration;
import alluxio.conf.ConfigurationValueOptions;
import alluxio.conf.PropertyKey;
import alluxio.conf.ReconfigurableRegistry;
import alluxio.conf.Source;
import alluxio.exception.AlluxioException;
import alluxio.exception.status.NotFoundException;
Expand Down Expand Up @@ -662,6 +663,9 @@ public Map<String, Boolean> updateConfiguration(Map<String, String> propertiesMa
}
}
LOG.debug("Update {} properties, succeed {}.", propertiesMap.size(), successCount);
if (successCount > 0) {
ReconfigurableRegistry.update();
}
return result;
}

Expand Down

0 comments on commit 407cf15

Please sign in to comment.