Re: RFR 8241078: OOM error parsing HTML with large Tag text

2020-03-18 Thread Alexey Ivanov
Hi Prasanta, Looks good to me too. On 18/03/2020 21:59, Sergey Bylokhov wrote: Looks fine. On 3/17/20 8:54 pm, Prasanta Sadhukhan wrote: Hi Sergey, Test updated http://cr.openjdk.java.net/~psadhukhan/8241078/webrev.1/ Regards Prasanta On 17-Mar-20 11:48 PM, Sergey Bylokhov wrote: HI,

Re: RFR 8241078: OOM error parsing HTML with large Tag text

2020-03-18 Thread Sergey Bylokhov
Looks fine. On 3/17/20 8:54 pm, Prasanta Sadhukhan wrote: Hi Sergey, Test updated http://cr.openjdk.java.net/~psadhukhan/8241078/webrev.1/ Regards Prasanta On 17-Mar-20 11:48 PM, Sergey Bylokhov wrote: HI, Prasanta. I suggest to limit memory in the test by the "-mx", otherwise the test

Re: RFR 8241078: OOM error parsing HTML with large Tag text

2020-03-17 Thread Prasanta Sadhukhan
Hi Sergey, Test updated http://cr.openjdk.java.net/~psadhukhan/8241078/webrev.1/ Regards Prasanta On 17-Mar-20 11:48 PM, Sergey Bylokhov wrote: HI, Prasanta. I suggest to limit memory in the test by the "-mx", otherwise the test will pass on the systems which has the big amount of memory. I

Re: RFR 8241078: OOM error parsing HTML with large Tag text

2020-03-17 Thread Sergey Bylokhov
HI, Prasanta. I suggest to limit memory in the test by the "-mx", otherwise the test will pass on the systems which has the big amount of memory. I tested 100G, when it pass. On 3/17/20 6:54 am, Prasanta Sadhukhan wrote: Hi All, Please review a fix for an issue where it is seen that OOM

RFR 8241078: OOM error parsing HTML with large Tag text

2020-03-17 Thread Prasanta Sadhukhan
Hi All, Please review a fix for an issue where it is seen that OOM errors generated when parsing HTML files with large tagtext. This is because of fix done in JDK-8173123 where the entire data array was copied again and again, insteadof the range specified via offset. Fix is to use the