Sure, but imagine, for example, given a production class with a single method, 
I add 3 methods which are used only in tests, the test will look more succinct, 
but the class won't. The coder who needs to understand the logic of the whole 
component needs to check usages of all 4 methods and scan through them to 
confirm that those 3 used only in tests and are not called in production and 
shouldn't be followed to understand how the component operates - waste of time 
and concentration, I think.

[ Full content available at: 
https://github.com/apache/incubator-druid/pull/6349 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to