fluo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From keith-turner <...@git.apache.org>
Subject [GitHub] incubator-fluo issue #768: Consider using SimpleConfiguration for per observ...
Date Wed, 14 Sep 2016 22:27:07 GMT
GitHub user keith-turner opened an issue:

    https://github.com/apache/incubator-fluo/issues/768

    Consider using SimpleConfiguration for per observer cofiguration.

    While writing the Fluo Tour and documenting application and observer configuration, I
noticed the two are not consistently designed.  Application config use the SimpleConfiguration
interface.  Per observer configuration uses a `Map<String, String>` and calls the config
parameters instead of configuration.
    
    These APIs evolved like this over time.  Writing the documentation made me notice the
divergent paths.  If both of these APIs had been designed at the same time, it likely would
have been more consistent.
    
    I think it would be nice of per observer config used SimpleConfiguration.  Wondering if
this should be made consistent before 1.0.0 or left as-is and made consistent later by adding
new methods.

----

----


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message