-
Notifications
You must be signed in to change notification settings - Fork 233
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
First attempt to OSGI-fy Kundera #525
base: trunk
Are you sure you want to change the base?
Conversation
Thanks for your contribution. I will have a look and merge it. Cheers, |
One more important note: JTA transactions will not work as of now. Has to do with java:comp naming space not being available in OSGI environment. |
This needs to be fixed. Technically this means a separate bundle for each client type and one for core. Finally add each as a module with module loader (say add with in modules with JBoss) with jar reference? To start with change looks ok. But this require a bit more work for completion Thanks. I will analyze and post an update on this. -Vivek |
If I have time, I might have a look at fixing the JTA transactions (will require me to dive into javax.naming a bit) and update pom files for all clients, just as was done for Mongo. not sure what you mean by modules... I don't really have much experience with JBoss. Can share the features for the Karaf though, which makes installation of Kundera pretty simple. |
Thanks. Please share your thoughts on Karaf. @dzalbo Would request you to spend few minutes to fill in for kundera survey: Cheers, |
@mevivs I've added another fix to this pull request... unrelated to the OSGI though.. it's a bug with MongoDBQuery. maxResult was lost for cases, when orderBy was present... |
I have just came across kundera and cassandra deployment in OSGi env. using my own patches as i haven't seen @dzalbo patches before. But 1st look at @dzalbo patches seems to solve OSGi env problems. |
@dzalbo
Does it something to do with java version? As i am building it with jdk 1.7 Am i missing anything? -Vivek |
@mevivs The error log most probably suggests that though com.impetus.kundera.osgi.Activator implements org.osgi.framework.BundleActivator, it is not of the same version JBoss expects. |
JBoss last release, I believe, supports OSGI 4.2.0, while in the pom.xml I suggested, I used 4.3.1. Maybe, decreasing the version will help. |
@dzalbo
is bit strange. -Vivek |
Did you install any OSGI bundles into JBoss? |
Trying to install kundera-core as an OSGI bundle -Vivek |
pom.xml got a bit messed up because of whitespaces...
What I basically did is added maven-bundle-plugin to generate OSGI Manifest and maven-jar-plugin to include this Manifest. I've done it now only for core module and mongo client. WIll need to be replicated for the rest
In the Core module, I added OSGI Activator and modified PersistenceUnitConfiguration to be able to load persistence.xml from other bundles.