[GitHub] activemq-artemis issue #672: [ARTEMIS-642] Disable slow client reconnecting ...
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/672 This is actually well put together now.. merging it. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] activemq-artemis issue #672: [ARTEMIS-642] Disable slow client reconnecting ...
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/672 The tests are all ok. I will do one final review tomorrow, but it looks good to merge. nice job! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] activemq-artemis issue #672: [ARTEMIS-642] Disable slow client reconnecting ...
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/672 I will run tests and merge this tomorrow. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] activemq-artemis issue #672: [ARTEMIS-642] Disable slow client reconnecting ...
Github user johnament commented on the issue: https://github.com/apache/activemq-artemis/pull/672 Looks good to me. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] activemq-artemis issue #672: [ARTEMIS-642] Disable slow client reconnecting ...
Github user bayern39 commented on the issue: https://github.com/apache/activemq-artemis/pull/672 @clebertsuconic I have done it ! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] activemq-artemis issue #672: [ARTEMIS-642] Disable slow client reconnecting ...
Github user bayern39 commented on the issue: https://github.com/apache/activemq-artemis/pull/672 @johnament OK,I have done it! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] activemq-artemis issue #672: [ARTEMIS-642] Disable slow client reconnecting ...
Github user johnament commented on the issue: https://github.com/apache/activemq-artemis/pull/672 @clebertsuconic think we can add the version aware stuff to the hacking guide? Would be useful for those not very familiar to know how to deal with it. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] activemq-artemis issue #672: [ARTEMIS-642] Disable slow client reconnecting ...
Github user bayern39 commented on the issue: https://github.com/apache/activemq-artemis/pull/672 @johnament @clebertsuconic Hi,I have fixed the mistake!Thanks! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] activemq-artemis issue #672: [ARTEMIS-642] Disable slow client reconnecting ...
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/672 Change activemq.version.incrementingVersion.. add 128 there... on activemq-version.properties.. add 128 to the compatibleVersionList also... When you are about to send the packet from CoreRemotingConenction... send the packet only if version >= 128.. make a constant on the packet for VERSION_INTRODUCED.. or something like that. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] activemq-artemis issue #672: [ARTEMIS-642] Disable slow client reconnecting ...
Github user bayern39 commented on the issue: https://github.com/apache/activemq-artemis/pull/672 @clebertsuconic Hi Clebertï¼I know how to get client version,but i do not know how to get supported versionï¼can you tell me more detailï¼Thanks --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] activemq-artemis issue #672: [ARTEMIS-642] Disable slow client reconnecting ...
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/672 This is much better... now the only thing wrong is the versioning. There's no need to add any fields. When the client connects to the server it will set the versionID when the session is created. You just need to increase the versionID. Check if the client.version is >= the supported version (You will need to create some constant for that on the Packet.. liker version created or so)... and only send the kill if the client understands this command. This is important not only for legacy support, but even to support 1.0, 1.1, 1.2 and 1.3 when we release 1.4+. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] activemq-artemis issue #672: [ARTEMIS-642] Disable slow client reconnecting ...
Github user bayern39 commented on the issue: https://github.com/apache/activemq-artemis/pull/672 @clebertsuconic Hi,Clebert,I have committed a new,I have done as you say, add a check of version ID and don't store the sessionContexts,but the last you sayed "rough the ProtocolManagers somehow",I try to do it,but it maybe inconvenient and difficult, so in order to not do a protocol specific implementation on QueueImpl,I have add a new method in the Interface of RemotingConnection,this is my solution,what do you think about it? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] activemq-artemis issue #672: [ARTEMIS-642] Disable slow client reconnecting ...
Github user bayern39 commented on the issue: https://github.com/apache/activemq-artemis/pull/672 @johnament I have fixed the mistake! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] activemq-artemis issue #672: [ARTEMIS-642] Disable slow client reconnecting ...
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/672 I don't think we can merge this... this is introducing a new datatype through the protocol and it will break compatibility with older clients in both directions. You would need to add handchecks for the version ID on the protocol if that's possible or not. I wouldn't do a hashMap storing sessions. I would just disable reconnections or kill all the sessions. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] activemq-artemis issue #672: [ARTEMIS-642] Disable slow client reconnecting ...
Github user bayern39 commented on the issue: https://github.com/apache/activemq-artemis/pull/672 @johnament yeah,I have seen the output,but when I run the new single test on my local machine,it is not wrong.I will run it more times. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] activemq-artemis issue #672: [ARTEMIS-642] Disable slow client reconnecting ...
Github user johnament commented on the issue: https://github.com/apache/activemq-artemis/pull/672 BTW, you may want to look at the jenkins output. May be a new test failing. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] activemq-artemis issue #672: [ARTEMIS-642] Disable slow client reconnecting ...
Github user johnament commented on the issue: https://github.com/apache/activemq-artemis/pull/672 @bayern39 sorry, you're right. I just realized your changes in `failoverOrReconnect` is in the client side. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] activemq-artemis issue #672: [ARTEMIS-642] Disable slow client reconnecting ...
Github user bayern39 commented on the issue: https://github.com/apache/activemq-artemis/pull/672 @johnament Thanks ,John,I will fix the mistake.The last one you have sayed,I do't understand it?The client judge the message type and decide the next step, it has been done. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] activemq-artemis issue #672: [ARTEMIS-642] Disable slow client reconnecting ...
Github user johnament commented on the issue: https://github.com/apache/activemq-artemis/pull/672 It strikes me that the client needs to know about this message type as well, potentially. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---