You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@oozie.apache.org by Peter Bacsko via Review Board <no...@reviews.apache.org> on 2018/05/15 13:56:54 UTC
Review Request 67132: OOZIE-3246 Flaky test
TestJMSJobEventListener#testConnectionDrop
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/67132/
-----------------------------------------------------------
Review request for oozie, András Piros and Peter Cseh.
Repository: oozie-git
Description
-------
Fix flaky test TestJMSJobEventListener#testConnectionDrop.
BrokerService instance must be closed properly.
Diffs
-----
core/src/test/java/org/apache/oozie/jms/TestJMSJobEventListener.java f375dec27
Diff: https://reviews.apache.org/r/67132/diff/1/
Testing
-------
Thanks,
Peter Bacsko
Re: Review Request 67132: OOZIE-3246 Flaky test
TestJMSJobEventListener#testConnectionDrop
Posted by András Piros via Review Board <no...@reviews.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/67132/#review203119
-----------------------------------------------------------
Ship it!
Ship It!
- András Piros
On May 15, 2018, 1:56 p.m., Peter Bacsko wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/67132/
> -----------------------------------------------------------
>
> (Updated May 15, 2018, 1:56 p.m.)
>
>
> Review request for oozie, András Piros and Peter Cseh.
>
>
> Repository: oozie-git
>
>
> Description
> -------
>
> Fix flaky test TestJMSJobEventListener#testConnectionDrop.
>
> BrokerService instance must be closed properly.
>
>
> Diffs
> -----
>
> core/src/test/java/org/apache/oozie/jms/TestJMSJobEventListener.java f375dec27
>
>
> Diff: https://reviews.apache.org/r/67132/diff/1/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Peter Bacsko
>
>