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 2022/03/29 08:51:02 UTC
[GitHub] [camel] orpiske opened a new pull request #7281: DO NOT MERGE: CAMEL-17589 WIP
orpiske opened a new pull request #7281:
URL: https://github.com/apache/camel/pull/7281
This is a WIP and available for the user to help diagnose and fix the
issue
<!-- Uncomment and fill this section if your PR is not trivial
- [ ] Make sure there is a [JIRA issue](https://issues.apache.org/jira/browse/CAMEL) filed for the change (usually before you start working on it). Trivial changes like typos do not require a JIRA issue. Your pull request should address just this issue, without pulling in other changes.
- [ ] Each commit in the pull request should have a meaningful subject line and body.
- [ ] If you're unsure, you can format the pull request title like `[CAMEL-XXX] Fixes bug in camel-file component`, where you replace `CAMEL-XXX` with the appropriate JIRA issue.
- [ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
- [ ] Run `mvn clean install -Psourcecheck` in your module with source check enabled to make sure basic checks pass and there are no checkstyle violations. A more thorough check will be performed on your pull request automatically.
Below are the contribution guidelines:
https://github.com/apache/camel/blob/main/CONTRIBUTING.md
-->
--
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.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel] orpiske merged pull request #7281: CAMEL-17859: increase logs in critical smpp code for diagnosing the issue root cause
Posted by GitBox <gi...@apache.org>.
orpiske merged pull request #7281:
URL: https://github.com/apache/camel/pull/7281
--
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.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel] orpiske commented on pull request #7281: DO NOT MERGE: CAMEL-17859 WIP
Posted by GitBox <gi...@apache.org>.
orpiske commented on pull request #7281:
URL: https://github.com/apache/camel/pull/7281#issuecomment-1083365368
> Just an FYI, I think the issue is [CAMEL-17859](https://issues.apache.org/jira/browse/CAMEL-17859) and not 17589.
Yeap, thanks!
--
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.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel] orpiske commented on pull request #7281: DO NOT MERGE: CAMEL-17589 WIP
Posted by GitBox <gi...@apache.org>.
orpiske commented on pull request #7281:
URL: https://github.com/apache/camel/pull/7281#issuecomment-1081604971
c/c: @PascalSchumacher if you can give it a try and see if we can collect more information with these ... Thanks in advance.
--
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.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel] PascalSchumacher commented on pull request #7281: CAMEL-17859: increase logs in critical smpp code for diagnosing the issue root cause
Posted by GitBox <gi...@apache.org>.
PascalSchumacher commented on pull request #7281:
URL: https://github.com/apache/camel/pull/7281#issuecomment-1085050789
Yes, the logging should make it easier to diagnose the problem if it occurs again.
--
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.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel] PascalSchumacher commented on pull request #7281: DO NOT MERGE: CAMEL-17859 WIP
Posted by GitBox <gi...@apache.org>.
PascalSchumacher commented on pull request #7281:
URL: https://github.com/apache/camel/pull/7281#issuecomment-1084758916
Sorry I am not able to reduce the issue. 😞
I am able to simulate part of the events from the log in the issue (existing connection, enquire link timeout) using a hacked copy of `DefaultSmppServer` from https://github.com/fizzed/cloudhopper-smpp.
But I do not know how to simulate `SocketException` (and the `InterruptedException` with the right timing).
--
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.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel] orpiske commented on pull request #7281: DO NOT MERGE: CAMEL-17859 WIP
Posted by GitBox <gi...@apache.org>.
orpiske commented on pull request #7281:
URL: https://github.com/apache/camel/pull/7281#issuecomment-1084798990
> Sorry I am not able to reduce the issue. 😞
>
> I am able to simulate part of the events from the log in the issue (existing connection, enquire link timeout) using a hacked copy of `DefaultSmppServer` from https://github.com/fizzed/cloudhopper-smpp.
>
> But I do not know how to simulate `SocketException` (and the `InterruptedException` with the right timing).
That's OK. For now, let's increase the logs in the `tryLock` failures so we are not in the dark in the case it fails. Based on our discussion on the ticket, I also believe they could play a role in this, but I don't want to change the code without a solid evidence of the root cause - primarily to avoid making it even worse.
--
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.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel] klease commented on pull request #7281: DO NOT MERGE: CAMEL-17589 WIP
Posted by GitBox <gi...@apache.org>.
klease commented on pull request #7281:
URL: https://github.com/apache/camel/pull/7281#issuecomment-1083363084
Just an FYI, I think the issue is CAMEL-17859 and not 17589.
--
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.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel] orpiske commented on pull request #7281: DO NOT MERGE: CAMEL-17859 WIP
Posted by GitBox <gi...@apache.org>.
orpiske commented on pull request #7281:
URL: https://github.com/apache/camel/pull/7281#issuecomment-1084799285
And thanks for taking the time to try to reproduce it!
--
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.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel] PascalSchumacher commented on pull request #7281: DO NOT MERGE: CAMEL-17589 WIP
Posted by GitBox <gi...@apache.org>.
PascalSchumacher commented on pull request #7281:
URL: https://github.com/apache/camel/pull/7281#issuecomment-1082220121
Thanks! 👍 I will try to replicate the problem in a test environment, but I probably will not have time to do this before the day after tomorrow.
--
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.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org