[ 
https://issues.apache.org/jira/browse/AMQ-4123?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13482615#comment-13482615
 ] 

Timothy Bish commented on AMQ-4123:
-----------------------------------

I'm not opposed to you adding in the conversion to a KeepAliveInfo, just wanted 
to note that its not altogether necessary. 
                
> Improve MQTT Inactivity Monitoring
> ----------------------------------
>
>                 Key: AMQ-4123
>                 URL: https://issues.apache.org/jira/browse/AMQ-4123
>             Project: ActiveMQ
>          Issue Type: Bug
>          Components: MQTT, Transport
>    Affects Versions: 5.7.0
>            Reporter: Christian Posta
>            Priority: Minor
>             Fix For: 5.8.0
>
>
> * Keep Alive should be 1.5 * the keep alive value specified by client (the 
> grace period described in spec: 
> http://public.dhe.ibm.com/software/dw/webservices/ws-mqtt/mqtt-v3r1.html)
> * MQTTInactivityMonitor checking for KeepAliveInfo in the onCommand, but it 
> can never get a KeepAliveInfo as it is
> * What to do when user explicitly turns Inactivity Monitor off?
> * Should there be defaults to close potentially dead connections?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to