[8u40] Request for approval: 8064391: More thread safety problems in core reflection
Martin Buchholz
martinrb at google.com
Thu Nov 13 16:30:19 UTC 2014
On Thu, Nov 13, 2014 at 7:17 AM, Joel Borggrén-Franck
<joel.franck at oracle.com> wrote:
> Hi Martin,
>
> I still like to backport your patch as is, because:
>
> - It is small and "obviously correct"
> - Fixing an existing, but hard to repro race
> - I was assuming "Here are changes we made locally at Google" implied it
> was in production
Yes, those are correct.
> If any of this was incorrect, I agree we should wait and reiterate this
> some more on core-libs-dev. If my assumptions were correct I think this
> should go in as is, potentially followed by more fixes later on.
Yes, it is perfectly fine to commit these changes now. A reason to
wait might be to consolidate these fixes with additional possible
fixes. But they should surely make the next update releases - not
sure what your schedule is for that.
More information about the jdk8u-dev
mailing list