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 2023/01/03 12:18:43 UTC
[GitHub] [camel] SteinOv opened a new pull request, #8967: camel-plc4x: Connection now established in doStart() of Producer/Consumer instead of constructors
SteinOv opened a new pull request, #8967:
URL: https://github.com/apache/camel/pull/8967
I do not have a JIRA account yet, so I could not create a JIRA ticket. I did create a issue over on the plc4x GitHub: [plc4x-bug-713](https://github.com/apache/plc4x/issues/713).
I moved establishment of the connection to the doStart() method instead of the constructor of the Endpoint. See the issue above for more information.
--
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] chrisdutz commented on pull request #8967: camel-plc4x: Connection now established in doStart() of Producer/Consumer instead of constructors
Posted by GitBox <gi...@apache.org>.
chrisdutz commented on PR #8967:
URL: https://github.com/apache/camel/pull/8967#issuecomment-1369778554
These changes were initially made in the PLC4X project and I suggested to bring them here, as we just recently donated the component here ... so from the PLC4X side, we would approve the changes.
--
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] github-actions[bot] commented on pull request #8967: camel-plc4x: Connection now established in doStart() of Producer/Consumer instead of constructors
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on PR #8967:
URL: https://github.com/apache/camel/pull/8967#issuecomment-1369705402
:star2: Thank you for your contribution to the Apache Camel project! :star2:
:warning: Please note that the changes on this PR may be **tested automatically**.
If necessary Apache Camel Committers may access logs and test results in the job summaries!
--
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] davsclaus merged pull request #8967: camel-plc4x: Connection now established in doStart() of Producer/Consumer instead of constructors
Posted by GitBox <gi...@apache.org>.
davsclaus merged PR #8967:
URL: https://github.com/apache/camel/pull/8967
--
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] github-actions[bot] commented on pull request #8967: camel-plc4x: Connection now established in doStart() of Producer/Consumer instead of constructors
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on PR #8967:
URL: https://github.com/apache/camel/pull/8967#issuecomment-1369751738
### Components tested:
| Total | Tested | Failed :x: | Passed :white_check_mark: |
| --- | --- | --- | --- |
| 1 | 1 | 0 | 1 |
--
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