RFR(s): 8035000: TEST_BUG: remove ActivationLibrary.DestroyThread and have callers call rmid.destroy() instead

Stuart Marks stuart.marks at oracle.com
Mon Dec 1 18:50:06 UTC 2014

Hi all, any reviewers for this one?


On 11/24/14 6:26 PM, Stuart Marks wrote:
> Hi all,
> Here's another test cleanup fix. Basically this normalizes the shutdown/destroy
> policy for rmid processes that are started by RMI's test library in support of a
> few dozen of the RMI activation tests. The previous implementation was, well,
> confused, and it had a redundant timing loop that can be subsumed by other code
> in RMI's test library (JavaVM.waitFor).
> I've listed this as a "small" changeset even though lots of files have changed.
> In all but three or so of the files, the only change is from calling
> "ActivationLibrary.rmidCleanup(rmid)" to calling "rmid.cleanup()" which is a
> refactoring enabled by the cleanup. The real action is in the files
> test/java/rmi/testlibrary/ActivationLibrary.java and RMID.java.
> Webrev:
>      http://cr.openjdk.java.net/~smarks/reviews/8035000/webrev.0/
> Bug:
>      https://bugs.openjdk.java.net/browse/JDK-8035000
> Thanks,
> s'marks

More information about the core-libs-dev mailing list