Github user clebertsuconic commented on the issue:

    https://github.com/apache/activemq-artemis/pull/1687
  
    @andytaylor @tabish121 onError isn't needed on this case... 
IOCriticalListener would kick in before (which is kicked after that method 
somewhere through the native interface on AIO).
    
    
    @andytaylor I think your changes here are fine. It would be nice to see a 
test.. but it's fine..
    
    
    I think it's ok to merge this, assuming you ran the AMQP tests on the 
testsuite.


---

Reply via email to