RFR 8021820: Number of opened files used in select() is limited to 1024 [macosx]

[Jonathan Gibbons]

On 08/08/2013 01:54 AM, Chris Hegarty wrote:
> On 08/08/2013 03:11 AM, Jonathan Gibbons wrote:
>> On 08/07/2013 07:10 PM, Stuart Marks wrote:
>>> I'd suggest putting a comment at the top of the test class saying that
>>> this test *must* be run in othervm mode, to ensure that files are
>>> closed properly. That way, you can take out the cleanupFiles() method
>>> too, as well as avoiding lots of exception handling and related
>>> cleanup code.
>>
>> Huh?   What's wrong with
>>      @run main/othervm classname
>
> I think what Stuart is suggesting is that '@run main/othervm 
> classname' be added, but also an additional text comment explaining 
> why this test is required to be run in othervm mode.
>
> During various test cleanup efforts over the years I've come across 
> many tests that specify '@run main/othervm ...' without any obvious 
> reason why. It is as if it was done (incorrectly) by certain engineers 
> as a matter of style! Adding a small text comment can be helpful for 
> future archaeologists ;-)
>
> In this particular case, if a reliable test can be created that runs 
> only on Mac then that is fine. But if the test is not reliable, or 
> hogs too much resources, then I think such a source change is ok 
> without a test.
>
> -Chris.
>
>>
>> -- Jon

Yes, I agree with you and Stuart that comments explaining the use of 
"@run main/othervm" would be a good idea. Sorry for the confusion.

-- jon