Github user srdo commented on the issue:
https://github.com/apache/storm/pull/2155
@hmcl I'm not sure how we can do that. The fields in Builder are not
static, so if we move the default definitions there, we'd have to create a
builder and fish out the default values in the tests. The defaults also
wouldn't be visible on the KafkaSpoutConfig class, which would probably be a
minus. I don't see a way to have the defaults solely in the KafkaSpoutConfig
class with no references to them from Builder, since the Builer fields have to
get the default values if nothing else is set.
We could move the FirstPollOffsetStrategy default into a static field in
KafkaSpoutConfig like the rest for consistency at least?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
