[GitHub] incubator-brooklyn pull request: Fix concurrent use of entity.setA...

2015-12-17 Thread aledsage
Github user aledsage commented on the pull request: https://github.com/apache/incubator-brooklyn/pull/1110#issuecomment-165446246 @ahgittin thanks. Interesting we weren't getting any `ConcurrentModificationException`s that I noticed. Instead we just had some subset of the 100

[GitHub] incubator-brooklyn pull request: Fix concurrent use of entity.setA...

2015-12-17 Thread sjcorbett
Github user sjcorbett commented on a diff in the pull request: https://github.com/apache/incubator-brooklyn/pull/1110#discussion_r47898623 --- Diff: core/src/main/java/org/apache/brooklyn/core/sensor/AttributeMap.java --- @@ -59,7 +60,20 @@ * Creates a new AttributeMap.

[GitHub] incubator-brooklyn pull request: Fix concurrent use of entity.setA...

2015-12-15 Thread aledsage
GitHub user aledsage opened a pull request: https://github.com/apache/incubator-brooklyn/pull/1110 Fix concurrent use of entity.setAttribute() Previously we were using a vanilla LinkedHashMap for storing attributes (ever since the