[
https://issues.apache.org/jira/browse/KAFKA-1040?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Jason Gustafson updated KAFKA-1040:
-----------------------------------
Fix Version/s: (was: 0.10.1.0)
> ConsumerConfig and ProducerConfig do "work" in the Constructor
> --------------------------------------------------------------
>
> Key: KAFKA-1040
> URL: https://issues.apache.org/jira/browse/KAFKA-1040
> Project: Kafka
> Issue Type: Improvement
> Components: config, consumer, producer
> Affects Versions: 0.8.0
> Environment: Java 1.7
> Linux Mint 14 (64bit)
> Reporter: Sharmarke Aden
> Assignee: Neha Narkhede
> Priority: Minor
> Labels: config, newbie
>
> It appears that validation of configuration properties is performed in the
> ConsumerConfig and ProducerConfig constructors. This is generally bad
> practice as it couples object construction and validation. It also makes it
> difficult to mock these objects in unit tests.
> Ideally validation of the configuration properties should be separated from
> object construction and initiated by those that rely/use these config objects.
> http://misko.hevery.com/code-reviewers-guide/flaw-constructor-does-real-work/
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
