Return-Path: X-Original-To: apmail-groovy-dev-archive@minotaur.apache.org Delivered-To: apmail-groovy-dev-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 413CC1851B for ; Thu, 29 Oct 2015 19:52:47 +0000 (UTC) Received: (qmail 45304 invoked by uid 500); 29 Oct 2015 19:52:44 -0000 Delivered-To: apmail-groovy-dev-archive@groovy.apache.org Received: (qmail 45258 invoked by uid 500); 29 Oct 2015 19:52:43 -0000 Mailing-List: contact dev-help@groovy.incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@groovy.incubator.apache.org Delivered-To: mailing list dev@groovy.incubator.apache.org Received: (qmail 44909 invoked by uid 99); 29 Oct 2015 19:52:43 -0000 Received: from Unknown (HELO spamd1-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 29 Oct 2015 19:52:43 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd1-us-west.apache.org (ASF Mail Server at spamd1-us-west.apache.org) with ESMTP id 3FA79C4B74; Thu, 29 Oct 2015 19:52:43 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd1-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: -0.002 X-Spam-Level: X-Spam-Status: No, score=-0.002 tagged_above=-999 required=6.31 tests=[RCVD_IN_MSPIKE_H2=-0.001, SPF_PASS=-0.001] autolearn=disabled Received: from mx1-eu-west.apache.org ([10.40.0.8]) by localhost (spamd1-us-west.apache.org [10.40.0.7]) (amavisd-new, port 10024) with ESMTP id 8DKtmEASGHq7; Thu, 29 Oct 2015 19:52:34 +0000 (UTC) Received: from mout.gmx.net (mout.gmx.net [212.227.17.22]) by mx1-eu-west.apache.org (ASF Mail Server at mx1-eu-west.apache.org) with ESMTPS id CEA8F215F9; Thu, 29 Oct 2015 19:52:33 +0000 (UTC) Received: from [192.168.1.5] ([85.180.102.216]) by mail.gmx.com (mrgmx103) with ESMTPSA (Nemesis) id 0MfmZs-1a66hz29wq-00ND3F; Thu, 29 Oct 2015 20:52:27 +0100 Subject: Re: Groovy category throwing MissingMethodException and MissingPropertyException when using multiple threads To: dev@groovy.incubator.apache.org, users@groovy.incubator.apache.org References: <56311C00.9020702@gmx.net> <1BCD07F2-703A-4EA8-B5C9-FBC5B99DC116@cs.vassar.edu> From: Jochen Theodorou Message-ID: <563278FA.8070404@gmx.org> Date: Thu, 29 Oct 2015 20:52:26 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 MIME-Version: 1.0 In-Reply-To: <1BCD07F2-703A-4EA8-B5C9-FBC5B99DC116@cs.vassar.edu> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit X-Provags-ID: V03:K0:R1InAU8FqqXZzexc31cHumW/v3vNBIJzyleud5Bdpf/ky7/3HKM tPws8u4lZE36xK+f1D3mO7+G8zWgHIviF8MXi3ZNqsOsPkBwyj/ml/8Tec7u9nVz0Bw92ja +lN6l7WvA6wVqkXu1xfduJtbJ2Cez84TbAXUaq8aUErGscKd3kD4VwQKzWqIJ8B2ou4i5SE E4r97ffa2lvA4qCdzUljg== X-UI-Out-Filterresults: notjunk:1;V01:K0:bsaZzBQ7GeU=:uTn9U3bvaIzCCMp5oh4VuJ EGLf5Vp/Fnuv0GfIEDgB9vL8hy77NVpX2BQrwsLDai7LkG9b1N3vWeKxMnWJIKGxkHY9rfdTI eG43t5lDdqvfR7MCa1wX+SJhGYy5YdspWty2xDRwvBRyIANqjOPNkb6uEy1D7m9mYehZf1hBU 2HxfxMtKn0mNGmwbWk5AK/fW5KuqBQqqjte88N+2k7qk/vK19e+F7Cdemi98ZAKkllOL+68fc iOncLTNSMFBQXliWwCfh3TwLIrqnghM/DCX11IV2skJ04/onG0dw9bGdXonFPzHSvKU6SVJEu /uygxZArEvFZUtNtJneGwbfcW3N16/iDSbh+0d3+MLHG2vPgNJYt5PYUhqh0bzyGhcRN1g7FX Zzu97HDqWBOju7Wo5MI83dGpTLASjGL+sXLfSZNcwnKjsuOuPpzxTBzSSLUVZRPzU07VCbN6d D5g6Bb389/lCB8mRC10uT/IHEYe/hx9/zQE7HtDcNuVkO0YWBzpm1BmvroeBWv6MROiTSoleC /CPIArhN9bMNi5eD/JRV4p4hWJjwypsVwVK/O+oPQ1nKU/zrrJ2TFijXeTJQZIms6TeAnYXND d42rbxAJcrPCnXxVnFDhNq+hep9LjrNGvpb2Jz8WdmOM8Wf68Cghdq45W0ggQNbO1YrmpH39q dO1eZuo+dayPXOhTHQTn+/KOBThV9ItXW568RNxuFtUl5OSVEIKJpkQm7GRGzr+8LOKliHcnY eMkSmFnQwmA/X+hOSQvJRIhM4b+rTvRZiskHd8C/4mn10rUrydiWHiiN0/pfrJWmcHpLqJz3g kq+81dC On 29.10.2015 11:48, Suderman Keith wrote: > There are (at least) two race conditions in GroovyCategorySupport.java > that would explain the intermittent and hard to reproduce bugs. > > I was hoping to submit this as my first official pull request to the > Groovy project, but gradle has different ideas… > > However, the offending code is in on lines 75-76 and 99-100 where > atomicCategoryUsageCounter is used non-atomically. If no one beats me to > it I will submit a PR once I puzzle out why ./gradlew test is failing > for me. Can you explain the non-atomic usage and why it leads to a problem here? bye blackdrag