search

hstaudacher / osgi-jax-rs-connector

An OSGi - JAX-RS 2.0 Connector, software repository available on the link below
http://hstaudacher.github.io/osgi-jax-rs-connector
Other
190 stars 98 forks source link

Optimized all resource tracker to stop tracking services that are not re... #93

Closed kolkoo closed 9 years ago

kolkoo commented 9 years ago

...sources.Fixed a bug with HttpTracker removedService method not being called. Fixed a bug with application not being marked dirty when removing a resource.

Optimized Configuration,ServletConfiguration and ApplicationConfiguration updates to not recreate the JerseyContext but instead update the existing objects. Reworked/extended tests.

Signed-off-by: IVAN GEORGIEV ILIEV ivan.iliev@musala.com

hstaudacher commented 9 years ago

Thanks will review in a few days. We are currently in a release phase which does not allow me to spend time for this right now ;(

kolkoo commented 9 years ago

That's ok for now we are using a locally built version of the publisher. When this makes it to a release we'll adapt :)

hstaudacher commented 9 years ago

Don't worry. I will fix them during review ;)

hstaudacher commented 9 years ago

Sorry, could not manage to integrate it so far. I'm also on vacation fro 2 weeks now. Will do it right after I return. Sorry for the delay

hstaudacher commented 9 years ago

working on it today

kolkoo commented 9 years ago

ok let me know how it goes :)

hstaudacher commented 9 years ago

Merged with minor changes. Can you verify that the current master version works for you?

kolkoo commented 9 years ago

Hi, yes I will check it out by the end of this week.

kolkoo commented 9 years ago

Sorry for the delayed response, everything is looking good for us. I am now using a built version from the latest master - and have added some slf4j logging for debugging purposes to it and all is looking good. Maybe we can consider adding some logging to the connector or what is your opinion about that?

hstaudacher commented 9 years ago

Nope, no logging :). If needed users can use the Jersey tracing feature, see https://jersey.java.net/documentation/latest/user-guide.html#appendix-properties

kolkoo commented 9 years ago

Yes, I thought so :P

kaikreuzer commented 9 years ago

@hstaudacher Any plans to do a new release which includes these optimizations?

hstaudacher commented 9 years ago

Yes, I jus tneed to upgrade to Jersey 2.18. Probably next week I will find some time