honeymoose
/
OpenSearch
mirror of https://github.com/honeymoose/OpenSearch.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix UpdateSettingsIT#testUpdateMergeMaxThreadCount. 

Log the setting(s) that changed in the compound updater.
This commit is contained in:
Jim Ferenczi 2016-09-08 13:38:18 +02:00
parent f319545814
commit f761038dfd
1 changed files with 11 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
core/src/main/java/org/elasticsearch/common/settings/Setting.java
View File

@ -411,6 +411,12 @@ public class Setting<T> extends ToXContentToBytes {
@Override @Override
public void apply(Tuple<A, B> value, Settings current, Settings previous) { public void apply(Tuple<A, B> value, Settings current, Settings previous) {
if (aSettingUpdater.hasChanged(current, previous)) {
logger.info("updating [{}] from [{}] to [{}]", aSetting.key, aSetting.getRaw(previous), aSetting.getRaw(current));
}
if (bSettingUpdater.hasChanged(current, previous)) {
logger.info("updating [{}] from [{}] to [{}]", bSetting.key, bSetting.getRaw(previous), bSetting.getRaw(current));
}
consumer.accept(value.v1(), value.v2()); consumer.accept(value.v1(), value.v2());
} }
@ -591,9 +597,9 @@ public class Setting<T> extends ToXContentToBytes {
* Creates a setting which specifies a memory size. This can either be * Creates a setting which specifies a memory size. This can either be
* specified as an absolute bytes value or as a percentage of the heap * specified as an absolute bytes value or as a percentage of the heap
* memory. * memory.
* *
* @param key the key for the setting * @param key the key for the setting
* @param defaultValue the default value for this setting * @param defaultValue the default value for this setting
* @param properties properties properties for this setting like scope, filtering... * @param properties properties properties for this setting like scope, filtering...
* @return the setting object * @return the setting object
*/ */
@ -606,9 +612,9 @@ public class Setting<T> extends ToXContentToBytes {
* Creates a setting which specifies a memory size. This can either be * Creates a setting which specifies a memory size. This can either be
* specified as an absolute bytes value or as a percentage of the heap * specified as an absolute bytes value or as a percentage of the heap
* memory. * memory.
* *
* @param key the key for the setting * @param key the key for the setting
* @param defaultValue a function that supplies the default value for this setting * @param defaultValue a function that supplies the default value for this setting
* @param properties properties properties for this setting like scope, filtering... * @param properties properties properties for this setting like scope, filtering...
* @return the setting object * @return the setting object
*/ */
@ -620,7 +626,7 @@ public class Setting<T> extends ToXContentToBytes {
* Creates a setting which specifies a memory size. This can either be * Creates a setting which specifies a memory size. This can either be
* specified as an absolute bytes value or as a percentage of the heap * specified as an absolute bytes value or as a percentage of the heap
* memory. * memory.
* *
* @param key the key for the setting * @param key the key for the setting
* @param defaultPercentage the default value of this setting as a percentage of the heap memory * @param defaultPercentage the default value of this setting as a percentage of the heap memory
* @param properties properties properties for this setting like scope, filtering... * @param properties properties properties for this setting like scope, filtering...