Re: RFR: 8332631: Update nsk.share.jpda.BindServer to don't use finalization [v2]

2024-05-24 Thread Leonid Mesnik
> The BindServer starts several threads and opens streams. > > It registered them for cleanup using "Finalizer" from nsk.share.framework. > Currently, it cleanup resources during shutdown hook. > > This fix changes BindServer to explicitly close streams and finish threads > after test is comple

Re: RFR: 8332631: Update nsk.share.jpda.BindServer to don't use finalization [v2]

2024-05-24 Thread Leonid Mesnik
On Wed, 22 May 2024 16:39:34 GMT, Chris Plummer wrote: >> Leonid Mesnik has updated the pull request incrementally with three >> additional commits since the last revision: >> >> - Merge branch '8332631' of https://github.com/lmesnik/jdk into 8332631 >> - Update test/hotspot/jtreg/vmTestbase/

Re: RFR: 8332631: Update nsk.share.jpda.BindServer to don't use finalization

2024-05-23 Thread Serguei Spitsyn
On Tue, 21 May 2024 19:55:01 GMT, Leonid Mesnik wrote: > The BindServer starts several threads and opens streams. > > It registered them for cleanup using "Finalizer" from nsk.share.framework. > Currently, it cleanup resources during shutdown hook. > > This fix changes BindServer to explicitly

Re: RFR: 8332631: Update nsk.share.jpda.BindServer to don't use finalization

2024-05-22 Thread Chris Plummer
On Tue, 21 May 2024 19:55:01 GMT, Leonid Mesnik wrote: > The BindServer starts several threads and opens streams. > > It registered them for cleanup using "Finalizer" from nsk.share.framework. > Currently, it cleanup resources during shutdown hook. > > This fix changes BindServer to explicitly

RFR: 8332631: Update nsk.share.jpda.BindServer to don't use finalization

2024-05-21 Thread Leonid Mesnik
The BindServer starts several threads and opens streams. It registered them for cleanup using "Finalizer" from nsk.share.framework. Currently, it cleanup resources during shutdown hook. This fix changes BindServer to explicitly close streams and finish threads after test is completed. The excep