THIS IS A TEST INSTANCE. ALL YOUR CHANGES WILL BE LOST!!!!
...
- Deprecate getXXX(String key, XXX defaultValue) and setXXX(String key, XXX value), discussed in the preview thread, except the following:
- getString(String key, String defaultValue)
- setString(String key, String value)
- getBytes(String key, byte[] defaultValue)
- setBytes(String key, byte[] bytes)
- Discussed at this preview thread: https://lists.apache.org/thread/zzsf7glfcdjcjm1hfo1xdwc6jp37nb3m
getClass(String key,
Class<? extends T> defaultValue, ClassLoader classLoader)
- setClass(String key, Class<?> klazz)
- Update the comment in getString String value), getBytes(String key, byte[] String defaultValue) , and setBytessetString(String key, byte[] bytesString value) to encourage users to use ConfigOption.
...
Code Block | ||||
---|---|---|---|---|
| ||||
@Public public class Configuration extends ExecutionConfig.GlobalJobParameters implements IOReadableWritable, java.io.Serializable, Cloneable, ReadableConfig, WritableConfig { ... /** * Returns the value associated with the given key as a string. We encourage users and * developers to always use ConfigOption for getting the configurations if possible, for its * rich description, type, default-value and other supports. The string-key-based getter should * only be used when ConfigOption is not applicable, e.g., the key is programmatically generated * in runtime. * * @param key the key pointing to the associated value * @param defaultValue the default value which is returned in case there is no value associated * with the given key * @return the (default) value associated with the given key */ public String getString(String key, String defaultValue) { ... } /** * Adds the given key/value pair to the configuration object. We encourage users and developers * to always use ConfigOption for setting the configurations if possible, for its rich * description, type, default-value and other supports. The string-key-based setter should only * be used when ConfigOption is not applicable, e.g., the key is programmatically generated in * runtime. * * @param key the key of the key/value pair to be added * @param value the value of the key/value pair to be added */ public void setString(String key, String value) { ... } * Returns the value associated with the given key as a byte array. We encourage users and * developers to always use ConfigOption for getting the configurations if possible, for its * rich description, type, default-value and other supports. The string-key-based getter should * only be used when ConfigOption is not applicable, e.g., the key is programmatically generated * in runtime. * * @param key The key pointing to the associated value. * @param defaultValue The default value which is returned in case there is no value associated * with the given key. * @return the (default) value associated with the given key. */ public byte[] getBytes(String key, byte[] defaultValue) { ... } /** * Adds the given byte array to the configuration object. We encourage users and developers to * always use ConfigOption for setting the configurations if possible, for its rich description, * type, default-value and other supports. The string-key-based setter should only be used when * ConfigOption is not applicable, e.g., the key is programmatically generated in runtime. * * @param key The key under which the bytes are added. * @param bytes The bytes to be added. */ public void setBytes(String key, byte[] bytes) { ... } ... ... } |
2.2.2 Deprecate some unnecessary setXxx and getXxx methods in Configuration
...