You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@brooklyn.apache.org by ahgittin <gi...@git.apache.org> on 2017/11/08 09:09:54 UTC

[GitHub] brooklyn-server issue #883: Deprecates use of Task in ConfigKey

Github user ahgittin commented on the issue:

    https://github.com/apache/brooklyn-server/pull/883
  
    Looks good.  Merging.
    
    Agree re `set(ConfigKey<T>, TaskFactory<T>)`.  Would also be nice to warn but similarly we can do that in a separate PR, once we've removed other uses.  In particular Java calls to `DependentConfiguration` eg `attributeWhenReady` will make `Task` instances so that is an obvious item to fix.
    
    There is related work in #853 which should be built upon as this continues.


---