RFR(T): 8235356: [TESTBUG] Disable 'producer is alive' check in JFR TestCrossProcessStreaming

[Mikhailo Seledtsov]

Dan, thank you. The change has been pushed.

On 12/4/19, 12:34 PM, Daniel D. Daugherty wrote:
> Thumbs up. I agree that this is a trivial fix. Thanks for reducing the
> noise in the JDK14 CI!
>
> Dan
>
>
> On 12/4/19 3:29 PM, Mikhailo Seledtsov wrote:
>> Please review this change that disables a specific check/assertion in 
>> the JFR TestCrossProcessStreaming test, until the underlying issue is 
>> resolved. Underlying issue is: "8235206: JFR 
>> api.consumer.streaming.TestCrossProcessStreaming.java fails with 
>> RuntimeException: Too few events are delivered while producer is 
>> alive: expected that 1 <= 0" is resolved."
>>
>>     JBS: https://bugs.openjdk.java.net/browse/JDK-8235356
>>     Testing: ran the updated test: PASS
>>     Change:
>>
>> --- 
>> a/test/jdk/jdk/jfr/api/consumer/streaming/TestCrossProcessStreaming.java
>> +++ 
>> b/test/jdk/jdk/jfr/api/consumer/streaming/TestCrossProcessStreaming.java
>> @@ -126,8 +126,9 @@
>>
>>          // Expected that some portion of events emitted by the 
>> producer are delivered
>>          // to the consumer while producer is still alive, at least 
>> one event for certain.
>> -        Asserts.assertLTE(1, ce.whileProducerAlive.get(),
>> -                           "Too few events are delivered while 
>> producer is alive");
>> +        // Assertion below is disabled due to: JDK-8235206
>> +        // Asserts.assertLTE(1, ce.whileProducerAlive.get(),
>> +        //                   "Too few events are delivered while 
>> producer is alive");
>>      }
>>
>>
>> Thank you,
>> Misha
>>
>