Hi,

As part of fixing the checkstyle violations on storm-kafka-client, we'd
need to rename some methods on KafkaSpoutConfig. See
https://github.com/apache/storm/pull/2117. Does anyone have an opinion on
when/if it is acceptable to break the API for this reason?

I'd also like opinions on whether method names like setSslKeystore are
better than setSSLKeystore? The current checkstyle rules prohibit adjacent
capital letters in non-final names due to this bit of the checkstyle config.

<module name="AbbreviationAsWordInName">
            <property name="ignoreFinal" value="false"/>
            <property name="allowedAbbreviationLength" value="1"/>
</module>

Reply via email to