RFR: 8146986: JDI: Signature lookups for unprepared classes can take a long time

Jean Christophe Beyler jcbeyler at google.com
Thu Mar 21 19:39:36 UTC 2019


Hi Egor,

Looks good to me as well, nit would be that the rest of the file does not
add a new-line for a else if:

if () {
} else if () {
}

instead of what you added:

if () {
}
else if () {
}

(Not a Reviewer)

Thanks,
Jc


On Thu, Mar 21, 2019 at 11:31 AM serguei.spitsyn at oracle.com <
serguei.spitsyn at oracle.com> wrote:

> Hi Egor,
>
> The fix looks good to me.
>
> Thank you for taking care about it!
> I've targeted this enhancement to 13.
>
> How do you test it?
> It would be safe to run all JDI tests to make sure no new timeouts are
> observed.
>
> Thanks,
> Serguei
>
>
> On 3/21/19 10:19, Egor Ushakov wrote:
> > Hi all, please review the fix
> >
> > it disables caching for types when signature is not provided as
> > described in the bug (this is one of the proposed solutions in the bug).
> >
> > https://bugs.openjdk.java.net/browse/JDK-8146986
> > http://cr.openjdk.java.net/~eushakov/8146986/webrev.00/
> >
> > Thanks!
> >
>
>

-- 

Thanks,
Jc
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20190321/ef35096f/attachment.html>


More information about the serviceability-dev mailing list