Netflix / archaius

Library for configuration management API
Apache License 2.0
2.46k stars 485 forks source link

Update documentation for DynamicWatchedConfiguration #556

Closed pravsingh closed 1 year ago

pravsingh commented 6 years ago

per the code below:

/**
 * A configuration that waits for a watcher event from the specified config source.
 * 
 * The property values in this configuration will be changed dynamically at runtime if the value changes in the
 * underlying configuration source.
 * <p>
 * This configuration does not allow null as key or value and will throw NullPointerException when trying to add or set
 * properties with empty key or value.
 * 
 * @author cfregly
 */
@SuppressWarnings("unchecked")
public class DynamicWatchedConfiguration extends ConcurrentMapConfiguration implements WatchedUpdateListener {

The keys and values must not be null however the document further mentions about empty key or value. Does it mean both null & empty are not allowed? if so then why we don't allow empty values?

rgallardo-netflix commented 1 year ago

v1 is no longer maintained.