storm-dynamic-spout icon indicating copy to clipboard operation
storm-dynamic-spout copied to clipboard

Break DynamicSpout, KafkaConsumer, and SidelineSpout into separate modules

Open Crim opened this issue 8 years ago • 2 comments

To provide clear separation of concerns between the DynamicSpout framework, the 'Sideline Spout' implementation, and the 'Kafka' consumer implementation, we should separate these out into maven modules.

Crim avatar Oct 18 '17 09:10 Crim

@Crim I feel like the 0.9 milestone is a bit ambition, can we focus on production-readiness in 0.9 and concentrate more splitting in 0.10?

stanlemon avatar Oct 21 '17 14:10 stanlemon

I think after #65 the only remaining blocker for this is SpoutConfig contains a reference to Kafka Consumer to set the default consumer class. That would need to be removed.

Crim avatar Nov 24 '17 06:11 Crim