[
https://issues.apache.org/jira/browse/KAFKA-3786?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Guozhang Wang updated KAFKA-3786:
---------------------------------
Description:
Currently the {{AbstractConfig}}'s constructor accepts the passed property map
as well as the {{ConfigDef}}, and maintains the original map as well as the
parsed values together. Because of it, with hierarchical config passing like
{{StreamsConfig}}, the underlying configs will takes all the key-value pairs
when constructed and hence cause WARNING log output.
We can consider adding an overloaded constructor in {{AbstractConfig}} to only
retains its defined configs as in {{ConfigDef}}.
was:
Currently the {{AbstractConfig}}'s constructor accepts the passed property map
as well as the {{ConfigDef}}, and maintains the original map as well as the
parsed values together. Because of it, with hierarchical config passing like
{{StreamsConfig}}, the underlying configs will takes all the key-value pairs
when constructed and hence cause WARNING log output.
We can consider adding another static constructor in {{AbstractConfig}} to only
retains its defined configs as in {{ConfigDef}}.
> AbstractConfig to only retain defined ConfigDef when parsing property map
> -------------------------------------------------------------------------
>
> Key: KAFKA-3786
> URL: https://issues.apache.org/jira/browse/KAFKA-3786
> Project: Kafka
> Issue Type: Bug
> Reporter: Guozhang Wang
> Assignee: Guozhang Wang
>
> Currently the {{AbstractConfig}}'s constructor accepts the passed property
> map as well as the {{ConfigDef}}, and maintains the original map as well as
> the parsed values together. Because of it, with hierarchical config passing
> like {{StreamsConfig}}, the underlying configs will takes all the key-value
> pairs when constructed and hence cause WARNING log output.
> We can consider adding an overloaded constructor in {{AbstractConfig}} to
> only retains its defined configs as in {{ConfigDef}}.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
