[GitHub] groovy pull request #484: GROOVY-8067: Possible deadlock when creating new C...

2017-02-07 Thread jwagenleitner
Github user jwagenleitner closed the pull request at:

https://github.com/apache/groovy/pull/484


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


[GitHub] groovy pull request #484: GROOVY-8067: Possible deadlock when creating new C...

2017-01-29 Thread jwagenleitner
GitHub user jwagenleitner opened a pull request:

https://github.com/apache/groovy/pull/484

GROOVY-8067: Possible deadlock when creating new ClassInfo entries in the 
cache

While I have been able to replicate the deadlock between 
`GroovyClassValuePreJava7$Segment` and the `GlobalClassSet#add`, I have not 
directly observed one between the `modifiedExpandos` and the `GlobalClassSet`, 
but think that it would be good to isolate their reference processing too since 
both lock in their operations.

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

$ git pull https://github.com/jwagenleitner/groovy groovy8067

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

https://github.com/apache/groovy/pull/484.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 #484


commit 78f5aa0b5977919ba05dcad9fe8a7ee496abf2e8
Author: John Wagenleitner 
Date:   2017-01-29T18:26:43Z

GROOVY-8067: test for demo only (DO NOT COMMIT)

commit 58a27e7b1c437d436636658a5de9537fda5560d6
Author: John Wagenleitner 
Date:   2017-01-29T19:34:55Z

GROOVY-8067: Possible deadlock when creating new ClassInfo entries in the 
cache




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