[GitHub] brooklyn-server pull request #993: Prevent racing persistence

[ahgittin <gi...@git.apache.org>]

GitHub user ahgittin opened a pull request:

    https://github.com/apache/brooklyn-server/pull/993

    Prevent racing persistence

    This _might_ fix the cause of the failure observed after #988 , and so then reinstates it, effectively unreverting #990 .
    
    Worth having this be tested a few times in the PR to see whether it does the trick.
    
    I suspect the FunctionFeed which is being left around is being added in a window where the entity is shutting down.  Just a guess though, not high confidence.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/ahgittin/brooklyn-server prevent-racing-persistence

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/brooklyn-server/pull/993.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #993
    
----
commit 457901222b84a5993a20460293ea53e1c6372389
Author: Alex Heneveld <al...@...>
Date:   2018-09-14T23:36:33Z

    prevent potential leaking persists

commit adbe60a49d4b0197db519c15f3be3d51f3a3577c
Author: Aled Sage <al...@...>
Date:   2018-09-10T23:28:12Z

    Fix AutoScalerPolicyRebindTest: when highlights change, request re-persist

----


---

[GitHub] brooklyn-server pull request #993: Prevent racing persistence

[ahgittin <gi...@git.apache.org>]

Github user ahgittin closed the pull request at:

    https://github.com/apache/brooklyn-server/pull/993


---

[GitHub] brooklyn-server issue #993: Prevent racing persistence

[ahgittin <gi...@git.apache.org>]

Github user ahgittin commented on the issue:

    https://github.com/apache/brooklyn-server/pull/993
  
    does not fix it :( ... closing for now


---