[ 
https://issues.apache.org/jira/browse/SAMZA-40?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14167432#comment-14167432
 ] 

Chris Riccomini commented on SAMZA-40:
--------------------------------------

Regarding config vs. wiring, Storm seems to have a pretty nice split for this. 
Wiring is done via the a builder 
([TopologyBuilder|https://storm.incubator.apache.org/apidocs/backtype/storm/topology/TopologyBuilder.html]),
 and config is done via a 
[Config|https://storm.incubator.apache.org/apidocs/backtype/storm/Config.html] 
object.

I don't believe that we should have a topology builder, but a JobBuilder would 
allow us to do job/container/task-level wiring in code, while still having a 
Config object for "real" config parameters.

> Refactor Samza configuration
> ----------------------------
>
>                 Key: SAMZA-40
>                 URL: https://issues.apache.org/jira/browse/SAMZA-40
>             Project: Samza
>          Issue Type: Bug
>          Components: container
>    Affects Versions: 0.6.0
>            Reporter: Chris Riccomini
>              Labels: project
>
> Samza's configuration system has several problems that we need to resolved.
> * Want to auto-generate documentation based off of configuration.
> * Should support global defaults for a config property. Right now, we do 
> config.getFoo.getOrElse() everywhere.
> * Should validate config up front, rather than thrown runtime exceptions 
> randomly throughout the code.
> * We are mixing wiring and configuration together. How do other systems 
> handle this?
> * We have fragmented configuration (anybody can define configuration). How do 
> other systems handle this?
> * How to handle undefined configuration? How to make this interoperable with 
> both Java and Scala (i.e. should we support Option in Scala)?
> * Should remain immutable.
> * Should remove implicits. It's just confusing.
> * Do we want to support complex types (list, map) for values, not just String?
> We need a design proposal for this.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to