RFR 8193156 : Need to backout fixes for JDK-8058547, JDK-8055753, JDK-8085903

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

RFR 8193156 : Need to backout fixes for JDK-8058547, JDK-8055753, JDK-8085903

Ivan Gerasimov
Hello!

A regression caused by the recent fixes of the ProtectionDomain cache
was observed.

Thus, it is proposed to revert the corresponding changes for now.

Would you please approve this patch, which is actually an anti-delta of
three mentioned fixes?

BUGURL: https://bugs.openjdk.java.net/browse/JDK-8193156
WEBREV: http://cr.openjdk.java.net/~igerasim/8193156/00/webrev/

Thanks in advance!

--
With kind regards,
Ivan Gerasimov

Reply | Threaded
Open this post in threaded view
|

Re: RFR 8193156 : Need to backout fixes for JDK-8058547, JDK-8055753, JDK-8085903

Sean Mullan
Can you keep line 48, which was just a javadoc wording fix? Otherwise,
looks ok. Please re-open or file new bugs for JDK-8058547 and JDK-8055753.

Thanks,
Sean

On 12/6/17 8:35 PM, Ivan Gerasimov wrote:

> Hello!
>
> A regression caused by the recent fixes of the ProtectionDomain cache
> was observed.
>
> Thus, it is proposed to revert the corresponding changes for now.
>
> Would you please approve this patch, which is actually an anti-delta of
> three mentioned fixes?
>
> BUGURL: https://bugs.openjdk.java.net/browse/JDK-8193156
> WEBREV: http://cr.openjdk.java.net/~igerasim/8193156/00/webrev/
>
> Thanks in advance!
>
Reply | Threaded
Open this post in threaded view
|

Re: RFR 8193156 : Need to backout fixes for JDK-8058547, JDK-8055753, JDK-8085903

Ivan Gerasimov
Thank you Sean for review!


On 12/7/17 6:10 AM, Sean Mullan wrote:
> Can you keep line 48, which was just a javadoc wording fix? Otherwise,
> looks ok.
Okay, I'll keep this change.

> Please re-open or file new bugs for JDK-8058547 and JDK-8055753.
>
Yes, will do.

With kind regards,
Ivan

> Thanks,
> Sean
>
> On 12/6/17 8:35 PM, Ivan Gerasimov wrote:
>> Hello!
>>
>> A regression caused by the recent fixes of the ProtectionDomain cache
>> was observed.
>>
>> Thus, it is proposed to revert the corresponding changes for now.
>>
>> Would you please approve this patch, which is actually an anti-delta
>> of three mentioned fixes?
>>
>> BUGURL: https://bugs.openjdk.java.net/browse/JDK-8193156
>> WEBREV: http://cr.openjdk.java.net/~igerasim/8193156/00/webrev/
>>
>> Thanks in advance!
>>
>

--
With kind regards,
Ivan Gerasimov