Re: RFR: 8264489: Add more logging to LargeCopyWithMark.java [v2]

2021-03-31 Thread Stefan Karlsson
On Wed, 31 Mar 2021 15:42:14 GMT, Brian Burkhalter wrote: >> Stefan Karlsson has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Review Alan > > Marked as reviewed by bpb (Reviewer). Thanks for reviewing! - PR: https://git.ope

Re: RFR: 8264489: Add more logging to LargeCopyWithMark.java [v2]

2021-03-31 Thread Brian Burkhalter
On Wed, 31 Mar 2021 11:15:43 GMT, Stefan Karlsson wrote: >> Add more logging to the LargeCopyWithMark.java test, to gather more >> information when this test fails with OOME. >> >> The intention is to gather more info for JDK-8239089. >> >> I consider this patch trivial, and expect to push it

Re: RFR: 8264489: Add more logging to LargeCopyWithMark.java [v2]

2021-03-31 Thread Alan Bateman
On Wed, 31 Mar 2021 11:15:43 GMT, Stefan Karlsson wrote: >> Add more logging to the LargeCopyWithMark.java test, to gather more >> information when this test fails with OOME. >> >> The intention is to gather more info for JDK-8239089. >> >> I consider this patch trivial, and expect to push it

Re: RFR: 8264489: Add more logging to LargeCopyWithMark.java [v2]

2021-03-31 Thread Stefan Karlsson
On Wed, 31 Mar 2021 09:29:43 GMT, Alan Bateman wrote: >> Stefan Karlsson has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Review Alan > > test/jdk/java/io/BufferedInputStream/LargeCopyWithMark.java line 29: > >> 27: * @summary BufferedI

Re: RFR: 8264489: Add more logging to LargeCopyWithMark.java [v2]

2021-03-31 Thread Stefan Karlsson
> Add more logging to the LargeCopyWithMark.java test, to gather more > information when this test fails with OOME. > > The intention is to gather more info for JDK-8239089. > > I consider this patch trivial, and expect to push it after I've gotten one > Review. Stefan Karlsson has updated th

Re: RFR: 8264489: Add more logging to LargeCopyWithMark.java

2021-03-31 Thread Alan Bateman
On Wed, 31 Mar 2021 06:56:23 GMT, Stefan Karlsson wrote: > Add more logging to the LargeCopyWithMark.java test, to gather more > information when this test fails with OOME. > > The intention is to gather more info for JDK-8239089. > > I consider this patch trivial, and expect to push it after

Re: RFR: 8264489: Add more logging to LargeCopyWithMark.java

2021-03-31 Thread Stefan Karlsson
On Wed, 31 Mar 2021 08:39:02 GMT, Kim Barrett wrote: >> Add more logging to the LargeCopyWithMark.java test, to gather more >> information when this test fails with OOME. >> >> The intention is to gather more info for JDK-8239089. >> >> I consider this patch trivial, and expect to push it aft

Re: RFR: 8264489: Add more logging to LargeCopyWithMark.java

2021-03-31 Thread Kim Barrett
On Wed, 31 Mar 2021 06:56:23 GMT, Stefan Karlsson wrote: > Add more logging to the LargeCopyWithMark.java test, to gather more > information when this test fails with OOME. > > The intention is to gather more info for JDK-8239089. > > I consider this patch trivial, and expect to push it after

RFR: 8264489: Add more logging to LargeCopyWithMark.java

2021-03-31 Thread Stefan Karlsson
Add more logging to the LargeCopyWithMark.java test, to gather more information when this test fails with OOME. The intention is to gather more info for JDK-8239089. I consider this patch trivial, and expect to push it after I've gotten on Review. - Commit messages: - 8264489: A