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
Add MongoDB extension #480
Add MongoDB extension #480
Conversation
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.
A question inline
@Produces | ||
@Singleton | ||
@Named("camelMongoClient") | ||
public com.mongodb.MongoClient camelMongoClient(MongoClient delegate) { |
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.
Can't we install the com.mongodb.MongoClient
into the Camel registry using MongoClientBuildItem
? Avoiding CDI could bring some performance benefits.
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.
I'll see if it's possible.
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.
Latest changes remove the CDI bits in favour of MongoClientBuildItem
.
095e359
to
873684a
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.
Very nice, thanks a lot!
ok to test |
Refer to this link for build results (access rights to CI server needed): |
fixes #302