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/10/04 15:05:10 UTC

[GitHub] brooklyn-server issue #835: Tasks subsystem improvements - adjuncts, names, ...

Github user ahgittin commented on the issue:

    https://github.com/apache/brooklyn-server/pull/835
  
    Completed this round of improvements.  A few more improvements:
    
    * many dynamic tasks run in same thread where possible, reducing overhead and supporting immediate execution
    * fix bug where subscriptions could be delivered out of order, where last subscription might not correspond to last value
    
    The last item, in https://github.com/apache/brooklyn-server/pull/835/commits/9213f0e25288114718f2a06a50ee206bf0120561, does a bit more synching so is slightly risky, but I've minimized this and I think it is the correct thing to do, consistent at least with other code paths.  Testing so far is good with it and there are ample notes on why.
    
    What I have _not_ done here is deprecation or warning in `getImmediately(Task)`, as suggested in #816.  That's turning in to a big job so I'll do it in another PR from which I'll reference this (so you should see a message about it below shortly.)


---