Skip to content

Conversation

momoysm
Copy link

@momoysm momoysm commented Aug 31, 2025

This pull request updates the camel.adoc documentation file.

Signed-off-by: SeongMo Yoon <momoysm@gmail.com>
@momoysm momoysm force-pushed the Update-camel.adoc branch from e49696f to fe3cc9d Compare August 31, 2025 15:01

To make the development process easier and to avoid unnecessary network hops, Apache Camel provides a https://camel.apache.org/components/3.18.x/spring-integration-component.html[module] to communicate with Spring Integration via message channels.
To make the development process easier and to avoid unnecessary network hops, Apache Camel provides a https://camel.apache.org/components/4.10.x/index.html[module] to communicate with Spring Integration via message channels.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I admit that I have missed to fix version for Apache Camel everywhere in this doc.
However, it looks like we have a bigger problem: Apache Camel does not provide Spring Integration module any more: https://camel.apache.org/manual/camel-4-migration-guide.html.

So, we need to revise this doc a bit more.
And as a subsequent task provide an Inbound Channel Adapter instead of the following sentence:

There is no inbound channel adapter because subscribing to a MessageChannel

And as a replacement for the removal of the whole component from Apache Camel.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your review!
I hadn't fully realized these issues. I really appreciate your feedback.

Would it be okay to close this PR?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Right. We will close today when we prepare a replacement PR with reality reflection.

@artembilan
Copy link
Member

Superseded with: #10375.

Thank you for your contribution!
It helped us to see more what is going on around.

@artembilan artembilan closed this Sep 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants