turcsanyip commented on pull request #4411: URL: https://github.com/apache/nifi/pull/4411#issuecomment-666975446
@tpalfy Removed the "poisoning approach", using conventional exception handling. There is one case (ConsumeJMS is idle/no messages in the queue and the broker shuts down) when there is no exception on the `onTrigger`'s thread. Introducing `isAlive()` method to detect it in `onTrigger`. `isAlive()` checks `connection.isOpen()` (so no explicit poisoning needed). ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
