You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by GitBox <gi...@apache.org> on 2021/04/22 07:55:56 UTC
[GitHub] [camel] mash-sap opened a new pull request #5453: CAMEL-16532: Prevent unregistration of existing endpoints after attem…
mash-sap opened a new pull request #5453:
URL: https://github.com/apache/camel/pull/5453
…pt to register a already existing endpoint
--
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:
users@infra.apache.org
[GitHub] [camel] mash-sap closed pull request #5453: CAMEL-16532: Prevent unregistration of existing endpoints after attem…
Posted by GitBox <gi...@apache.org>.
mash-sap closed pull request #5453:
URL: https://github.com/apache/camel/pull/5453
--
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:
users@infra.apache.org
[GitHub] [camel] mash-sap commented on pull request #5453: CAMEL-16532: Prevent unregistration of existing endpoints after attem…
Posted by GitBox <gi...@apache.org>.
mash-sap commented on pull request #5453:
URL: https://github.com/apache/camel/pull/5453#issuecomment-824646960
@tadayosi The change to destroy the server on stop was introduced to cleanup mbeans (CAMEL-10914). Can we check if the proposed change has a impact on that specific mbean-use-case? Do you know what would need to be done to check?
--
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:
users@infra.apache.org
[GitHub] [camel] mash-sap commented on pull request #5453: CAMEL-16532: Prevent unregistration of existing endpoints after attem…
Posted by GitBox <gi...@apache.org>.
mash-sap commented on pull request #5453:
URL: https://github.com/apache/camel/pull/5453#issuecomment-824675595
I was not able to update the branch. Therefore I created a new PR: https://github.com/apache/camel/pull/5454
--
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:
users@infra.apache.org
[GitHub] [camel] mash-sap commented on pull request #5453: CAMEL-16532: Prevent unregistration of existing endpoints after attem…
Posted by GitBox <gi...@apache.org>.
mash-sap commented on pull request #5453:
URL: https://github.com/apache/camel/pull/5453#issuecomment-824660295
@tadayosi How could I check for dangling mbeans?
--
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:
users@infra.apache.org
[GitHub] [camel] tadayosi commented on pull request #5453: CAMEL-16532: Prevent unregistration of existing endpoints after attem…
Posted by GitBox <gi...@apache.org>.
tadayosi commented on pull request #5453:
URL: https://github.com/apache/camel/pull/5453#issuecomment-824660702
You can check them by using some JMX viewers such as Hawtio.
--
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:
users@infra.apache.org
[GitHub] [camel] tadayosi commented on pull request #5453: CAMEL-16532: Prevent unregistration of existing endpoints after attem…
Posted by GitBox <gi...@apache.org>.
tadayosi commented on pull request #5453:
URL: https://github.com/apache/camel/pull/5453#issuecomment-824659412
And you should create the patch against {{main}} from scratch, otherwise it'll see merge conflicts like this.
--
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:
users@infra.apache.org
[GitHub] [camel] tadayosi commented on pull request #5453: CAMEL-16532: Prevent unregistration of existing endpoints after attem…
Posted by GitBox <gi...@apache.org>.
tadayosi commented on pull request #5453:
URL: https://github.com/apache/camel/pull/5453#issuecomment-824658179
@mash-sap Maybe you can try to make some CXF consumers fail to start and check if there are really no dangling MBeans for them.
--
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:
users@infra.apache.org