Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [eclipselink-dev] [Fwd: [Bug 238696] Fix use of Double Checked Locking pattern]

Hi Mitesh,

  I believe James has gotten back to you about the fix.

  I am wondering if your test failures are JDK 1.6-related.  See the following bug:

https://bugs.eclipse.org/bugs/show_bug.cgi?id=236185

There is a set of test results listed near the bottom that has a similar number of failures for JDK 1.6 compile and run scenarios.

-Tom

Mitesh Meswani wrote:
Please review attached patch.
Tests run: LRG at top level. No regression detected due to the patch (184 test are failing under core with or without changes due to this patch)

Thanks,
Mitesh

-------- Original Message --------
Subject:     [Bug 238696] Fix use of Double Checked Locking pattern
Date:     Wed, 23 Jul 2008 19:21:31 -0400
From:     bugzilla-daemon@xxxxxxxxxxx
To:     Mitesh.Meswani@xxxxxxx
References:     <bug-238696-44960@xxxxxxxxxxxxxxxxxxxxxx/bugs/>



https://bugs.eclipse.org/bugs/show_bug.cgi?id=238696 Product/Component: EclipseLink / Foundation

Mitesh Meswani <mitesh.meswani@xxxxxxx> changed:

          What    |Removed                     |Added
----------------------------------------------------------------------------
Attachment #107673|0                           |1
       is obsolete|                            |




--- Comment #7 from Mitesh Meswani <mitesh.meswani@xxxxxxx> 2008-07-23 19:21:29 -0400 ---
Created an attachment (id=108282)
--> (https://bugs.eclipse.org/bugs/attachment.cgi?id=108282)
Proposed patch to fix only the proven broaken code path

The attached patch eagerly initializes platform operators to eliminate
possibility of a race condition. Other code path are though theoretically
broken, are not considered to be a risk enough to introduce performance penalty
of using a volatile variable



Back to the top