-
Notifications
You must be signed in to change notification settings - Fork 4.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CAMEL-14633 #3603
CAMEL-14633 #3603
Conversation
…py without headers The new two tests will make a safe copy of the exchange. in both tests, the in and the out message have a attachment but in only one of the tests will set an header on the messages before copying the exchange. The test without headers will fail.
Moved logic to copy attachments outside of if-block to check for existence of headers
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please run the build with the profile sourcecheck enabled and fix the warning. It looks good to me otherwise.
Hi, are you referring to this line of the build output? [WARNING] No eip doc file: /Users/olimberg/projects/Java/camel/camel-core/src/main/docs/eips/pipeline-eip.adoc I'll try to figure out how to solve this issue. |
No, I'm referring on the warnings you should have when compiling with -Psourcecheck, you should see at least one error at the beginning related to imports order. |
} | ||
// just copy the attachments here |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As attachments are very very seldom in use, then this has negative performance impact. You should check via hasAttachments
if any copying is needed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay, should I apply this change to the out message as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah that would be good
…ng the copy method
No description provided.