Bug 64969 - RemoteJMeterEngineImpl#rexit didn't unexport result int the RemoteJMeterEngineImpl instance couldn't release
Summary: RemoteJMeterEngineImpl#rexit didn't unexport result int the RemoteJMeterEngin...
Status: RESOLVED FIXED
Alias: None
Product: JMeter
Classification: Unclassified
Component: Main (show other bugs)
Version: 3.0
Hardware: PC All
: P2 normal (vote)
Target Milestone: JMETER 5.4.1
Assignee: JMeter issues mailing list
URL:
Keywords: FixedInTrunk
Depends on:
Blocks:
 
Reported: 2020-12-09 05:16 UTC by 罗寅卓
Modified: 2020-12-09 12:15 UTC (History)
1 user (show)



Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description 罗寅卓 2020-12-09 05:16:17 UTC
The RemoteJMeterEngineImpl#rexit() method is supposed to close the RemoteJMeterEngineImpl instance. Although it has unbinded the RemoteJMeterEngineImpl instance from RMI Registry now, it hasn't invoked UnicastRemoteObject#unexportObject(Remote obj, boolean force) method to release the RemoteJMeterEngineImpl instance which result in the RemoteJMeterEngineImpl instance couldn't release and the RemoteJMeterEngineImpl instance's RemoteJMeterEngineImpl#localPort couldn't close too.

To solve this problem, it is suggested to add "UnicastRemoteObject#unexportObject(this, false)" after line 210 before "JMeterUtils.helpGC()" in "RemoteJMeterEngineImpl.java" file.
Comment 1 Philippe Mouawad 2020-12-09 12:15:16 UTC
pmouawad pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/jmeter.git


The following commit(s) were added to refs/heads/master by this push:
     new 5bb4fbe  Fix bug 64969, make RemoteJMeterEngineImpl instance release completely when RemoteJMeterEngineImpl#rexit() invoked
5bb4fbe is described below

commit 5bb4fbe389204290c61afbd12bac66a8ca928755
Author: pmouawad <p.mouawad@ubik-ingenierie.com>
AuthorDate: Wed Dec 9 09:21:21 2020 +0100

    Fix bug 64969, make RemoteJMeterEngineImpl instance release completely when RemoteJMeterEngineImpl#rexit() invoked
---
 .../java/org/apache/jmeter/engine/RemoteJMeterEngineImpl.java     | 8 ++++++++
 xdocs/changes.xml                                                 | 2 ++
 2 files changed, 10 insertions(+)