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-15689 - Add camel-atlasmap component #4447
Conversation
0816227
to
efeff50
Compare
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.
LGTM. Lets wait for others reviewers. Thanks @igarashitm !
if (!errors.isEmpty()) { | ||
StringBuilder buf = new StringBuilder("Errors: "); | ||
errors.stream().forEach(a -> buf.append( | ||
String.format("[%s: Document='{}(ID:{})', path='%s'], ", |
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.
Do we have a mix here where last 2 params are ignored ?
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.
Doh good catch, thanks! fixed.
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.
Just a small question above but otherwise looks good to me. Well done @igarashitm :)
efeff50
to
908a173
Compare
908a173
to
84ce099
Compare
[CAMEL-XXX] Fixes bug in camel-file component
, where you replaceCAMEL-XXX
with the appropriate JIRA issue.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/master/CONTRIBUTING.md