Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

...

Including the dataType could open up potentially other use cases as well such as TODO.

In this proposal we modify the DescribeConfigsResponse schema to include the field type. 

...

No new method will be added or updated to the AdminCient class. 

Proposed Changes

Under the proposed change, AdminClient.describeConfigs response (DescribeConfigsResponse.ConfigEntry) would include a new property Type. This directly maps to the Type enum as defined in ConfigDef.java

Code Block
 /**
 * The config types
 */
 public enum Type {
   BOOLEAN, STRING, INT, SHORT, LONG, DOUBLE, LIST, CLASS, PASSWORD
 }


DescribeConfigsResponse.ConfigEntry

Code Block
public class ConfigEntry {
    private final String name;
    private final String value;
    private final ConfigSource source;
    private final boolean isSensitive;
    private final boolean isReadOnly;
    private final String type; // This is the new property
    private final List<ConfigSynonym> synonyms;
.....

Rest of the changes would be plumbing this new property in AdminManager class to be eventually consumed by AdminClient class

Compatibility, Deprecation, and Migration Plan

...