On Tue, 15 Nov 2022 20:04:07 GMT, Alexander Zuev <[email protected]> wrote:

>> On m1 mac sometimes (in very rare instance) it takes a long time to get the 
>> sequencer so increasing timeout helps with that.
>> Also on m1 mac sequencer got notified about the first MetaMessage twice. The 
>> reason is unclear but watching for that fixes the problem. Also on Linux 
>> sometimes process hangs indefinitely without finishing hold by the 
>> com.sun.media.sound.MediaDispatcher. Closing the sequencer at the end of the 
>> test helps with that.
>
> Alexander Zuev has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Lower the timeout to 120 seconds

FWIW 120 is the default .. so that isn't a difference, but 240 seconds in CI is 
long enough
And I suspect that Sequencer.close() is going to help.

-------------

Marked as reviewed by prr (Reviewer).

PR: https://git.openjdk.org/jdk/pull/11157

Reply via email to