Re: RFR: 8311775: [TEST] duplicate verifyHeapDump in several tests [v5]

2023-09-01 Thread Chris Plummer
On Fri, 1 Sep 2023 02:00:04 GMT, Yi Yang wrote: >> This is a follow-up patch of #13667. verifyHeapDump is duplicated in several >> tests, this patch tries to consolidate them into one method. > > Yi Yang has updated the pull request incrementally with one additional commit > since the last revi

Re: RFR: 8311775: [TEST] duplicate verifyHeapDump in several tests [v5]

2023-09-01 Thread Alex Menkov
On Fri, 1 Sep 2023 02:00:04 GMT, Yi Yang wrote: >> This is a follow-up patch of #13667. verifyHeapDump is duplicated in several >> tests, this patch tries to consolidate them into one method. > > Yi Yang has updated the pull request incrementally with one additional commit > since the last revi

Re: RFR: 8311775: [TEST] duplicate verifyHeapDump in several tests [v5]

2023-08-31 Thread Yi Yang
> This is a follow-up patch of #13667. verifyHeapDump is duplicated in several > tests, this patch tries to consolidate them into one method. Yi Yang has updated the pull request incrementally with one additional commit since the last revision: Update HprofParser.java - Changes:

Re: RFR: 8311775: [TEST] duplicate verifyHeapDump in several tests [v4]

2023-08-31 Thread Chris Plummer
On Thu, 31 Aug 2023 01:53:36 GMT, Yi Yang wrote: >> This is a follow-up patch of #13667. verifyHeapDump is duplicated in several >> tests, this patch tries to consolidate them into one method. > > Yi Yang has updated the pull request incrementally with one additional commit > since the last rev

Re: RFR: 8311775: [TEST] duplicate verifyHeapDump in several tests [v4]

2023-08-30 Thread Yi Yang
> This is a follow-up patch of #13667. verifyHeapDump is duplicated in several > tests, this patch tries to consolidate them into one method. Yi Yang has updated the pull request incrementally with one additional commit since the last revision: Update HeapDumpTest.java - Changes

Re: RFR: 8311775: [TEST] duplicate verifyHeapDump in several tests [v3]

2023-08-28 Thread Alex Menkov
On Fri, 11 Aug 2023 02:46:36 GMT, Yi Yang wrote: >> This is a follow-up patch of #13667. verifyHeapDump is duplicated in several >> tests, this patch tries to consolidate them into one method. > > Yi Yang has updated the pull request incrementally with one additional commit > since the last rev

Re: RFR: 8311775: [TEST] duplicate verifyHeapDump in several tests [v2]

2023-08-10 Thread Yi Yang
On Thu, 10 Aug 2023 19:29:27 GMT, Alex Menkov wrote: >> Yi Yang has updated the pull request incrementally with one additional >> commit since the last revision: >> >> fix TestHeapDumpForInvokDynamic > > test/hotspot/jtreg/serviceability/sa/TestHeapDumpForInvokeDynamic.java line > 40: > >>

Re: RFR: 8311775: [TEST] duplicate verifyHeapDump in several tests [v3]

2023-08-10 Thread Yi Yang
> This is a follow-up patch of #13667. verifyHeapDump is duplicated in several > tests, this patch tries to consolidate them into one method. Yi Yang has updated the pull request incrementally with one additional commit since the last revision: review from Alex - Changes: - al

Re: RFR: 8311775: [TEST] duplicate verifyHeapDump in several tests [v2]

2023-08-10 Thread Yi Yang
On Thu, 10 Aug 2023 09:41:29 GMT, Yi Yang wrote: >> This is a follow-up patch of #13667. verifyHeapDump is duplicated in several >> tests, this patch tries to consolidate them into one method. > > Yi Yang has updated the pull request incrementally with one additional commit > since the last rev

Re: RFR: 8311775: [TEST] duplicate verifyHeapDump in several tests [v2]

2023-08-10 Thread Alex Menkov
On Thu, 10 Aug 2023 09:41:29 GMT, Yi Yang wrote: >> This is a follow-up patch of #13667. verifyHeapDump is duplicated in several >> tests, this patch tries to consolidate them into one method. > > Yi Yang has updated the pull request incrementally with one additional commit > since the last rev

Re: RFR: 8311775: [TEST] duplicate verifyHeapDump in several tests [v2]

2023-08-10 Thread Chris Plummer
On Thu, 10 Aug 2023 09:41:29 GMT, Yi Yang wrote: >> This is a follow-up patch of #13667. verifyHeapDump is duplicated in several >> tests, this patch tries to consolidate them into one method. > > Yi Yang has updated the pull request incrementally with one additional commit > since the last rev

Re: RFR: 8311775: [TEST] duplicate verifyHeapDump in several tests [v2]

2023-08-10 Thread Kevin Walls
On Thu, 10 Aug 2023 09:41:29 GMT, Yi Yang wrote: >> This is a follow-up patch of #13667. verifyHeapDump is duplicated in several >> tests, this patch tries to consolidate them into one method. > > Yi Yang has updated the pull request incrementally with one additional commit > since the last rev

Re: RFR: 8311775: [TEST] duplicate verifyHeapDump in several tests [v2]

2023-08-10 Thread Yi Yang
> This is a follow-up patch of #13667. verifyHeapDump is duplicated in several > tests, this patch tries to consolidate them into one method. Yi Yang has updated the pull request incrementally with one additional commit since the last revision: fix TestHeapDumpForInvokDynamic -

RFR: 8311775: [TEST] duplicate verifyHeapDump in several tests

2023-08-09 Thread Yi Yang
This is a follow-up patch of #13667. verifyHeapDump is duplicated in several tests, this patch tries to consolidate them into one method. - Commit messages: - 8311775: [TEST] duplicate verifyHeapDump in several tests Changes: https://git.openjdk.org/jdk/pull/15202/files Webrev: ht