Hi,

On 10/25/2016 4:31 PM, Paul Sandoz wrote:
Hi Roger,

Do you need to use a double-checked locking idiom in getEchoStub?
Probably not, I'm  just practicing the idiom.
We are unlikely to move to concurrent test execution in testng (when it supports it).

I cannot recall the life-cycle of per test execution, but i suspect you might 
be able to turn the static field and method getEchoStub into a non-static field 
and just make the method synchronized, but you might ned to annotate the 
shutdown with @AfterClass instead.
That would work too.

What's the rule for choosing synchronized vs volatile? I've gotten different recommendations in different cases.
What's Occam's razor here?

Actually, perhaps even better initiate the ORB and create the stub in a 
@BeforeClass method and clean up in a @AfterClass method?
that would work, but it is only used in one of the tests. So just trying to keep the usage localized.

Thanks, Roger


Paul.

On 25 Oct 2016, at 12:51, Roger Riggs <roger.ri...@oracle.com> wrote:

Please review a test fix for an address-in-use problem in this CORBA 
serialization test.
The test is simplified to just communicate client to server via the orb in the 
process.

It does not need activation, a name service, or the orbd to exercise the
necessary serialization implementation.  And therefore, no contention for 
sockets, ports, etc.

Webrev:
   http://cr.openjdk.java.net/~rriggs/webrev-orb-addrinuse-8168613/

Thanks, Roger


Reply via email to