Github user clebertsuconic commented on a diff in the pull request:

    https://github.com/apache/activemq-artemis/pull/2149#discussion_r196751676
  
    --- Diff: 
tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/amqp/AmqpNoHearbeatsTest.java
 ---
    @@ -80,4 +88,68 @@ public void inspectOpenedResource(Connection connection) 
{
           connection.close();
        }
     
    +   private static final String QUEUE_NAME = "queue://testHeartless";
    +
    +   // This test is validating a scenario where the client will leave with 
connection reset
    +   // This is done by setting soLinger=0 on the socket, which will make 
the system to issue a connection.reset instead of sending a
    +   // disconnect.
    +   @Test(timeout = 60000)
    +   public void testCloseConsumerOnConnectionReset() throws Exception {
    --- End diff --
    
    The two are related... you should be able to cleanup consumer, disable 
heartbeats and call disconnect on windows. ARTEMIS-1927 was found after I 
disabled heart beats and killed a consumer on windows which issued a connection 
reset.


---

Reply via email to