Hot deploying a module unnecessarily restarts all other modules

[jri]

This happens only if the deployed module provides REST resources.

Thanks dgf for pointing this out!

[Jörg Richter]

Core: use a Jersey servlet singleton (#272).

We hope this will solve the oddities with module hot deployment.

The singleton is encapsulated in a new OSGi "WebPublishingService?" as provided by the DM core.
The plugins register their resources through this service.

Not fully functional.

See ticket 272.

• Changeset: d87925be910399d851ff1bdda16632aa9c8ed6d1
• URL: ​https://github.com/jri/deepamehta/commit/d87925be910399d851ff1bdda16632aa9c8ed6d1

[Jörg Richter]

Core: adapt plugin REST services (#272).

A plugin's provider classes are picked up automatically.
A plugin must no longer provide an Application class.

The Webclient's start URL has changed:

​http://localhost:8080/de.deepamehta.webclient/

The root namespace is already occupied by the Jersey root application.

IMPORTANT: To adapt to the new Webclient base URL you must reset your database.

Not fully functional.

See ticket 272.

• Changeset: d6a40ff084ba4edd37774a6f9187e99686ebc863
• URL: ​https://github.com/jri/deepamehta/commit/d6a40ff084ba4edd37774a6f9187e99686ebc863

[Jörg Richter]

Core: WebPublishing? service basically works (#272)

Plugins add their REST resources via the WebPublishing? service.
Removing resources is pending.

Plugin's "Application" classes are dropped.

See ticket 272.

• Changeset: ec06c506990eda43bc2790fd70af4f80c520220d
• URL: ​https://github.com/jri/deepamehta/commit/ec06c506990eda43bc2790fd70af4f80c520220d

[Jörg Richter]

Core: removing REST resources dynamically (#272).

Reloading a module (via hot deployment) that provides REST resources works properly now. Other modules are not affected.
So, the main problem seems to be solved now.

Secondary issues remain (e.g. reloading the core, or shutdown) and will be addressed shortly.

Hi dgf! Your intuition with the multiple Jersey servlet instances as the culprit seems to prove correct.
Thank you!

See ticket 272.

• Changeset: f49829541c0a3c9203e829690cc867360d852901
• URL: ​https://github.com/jri/deepamehta/commit/f49829541c0a3c9203e829690cc867360d852901

[Jörg Richter]

Core: dynamic un/register Jersey servlet (#272).

Once all REST resources are removed the Jersey servlet is unregistered.
As soon as resources are added the Jersey servlet is registered again.
So, actions like reloading the DM core or shutdown the platform perform properly.

Note: there is still an issue with reloading the Webclient module but this is not REST related and needs to be addressed in another ticket.

See ticket 272.

• Changeset: b0f31852d78b883d21138f024b4c3864b6d9e6cc
• URL: ​https://github.com/jri/deepamehta/commit/b0f31852d78b883d21138f024b4c3864b6d9e6cc

[Jörg Richter]

Core: abolish resource config properties (#272).

2 properties are no longer required to be set in plugin.properties:

"restResourcesNamespace" -- now set in the plugin class's @Path anotation
"webResourcesNamespace" -- not configurable anymore

See ticket 272.

• Changeset: a39e39c19b1d0c46eec8f9abe054d7d2255b5a48
• URL: ​https://github.com/jri/deepamehta/commit/a39e39c19b1d0c46eec8f9abe054d7d2255b5a48

[Jörg Richter]

Core: plugins doesn't consume HttpService? (#272).

Instead they are register their web resources through the WebPublishingService?.

WebPublishingService? provides 2 new methods:

WebResources addWebResources(Bundle bundle, String uriNamespace)
void removeWebResources(WebResources webResources)

All web publishing features are encapsulated in the WebPublishingService?.

See ticket 272.

• Changeset: e47c8507aba7db17271d3ab3a6144944fdbdd522
• URL: ​https://github.com/jri/deepamehta/commit/e47c8507aba7db17271d3ab3a6144944fdbdd522

[Jörg Richter]

Core: use a Jersey servlet singleton (#272).

We hope this will solve the oddities with module hot deployment.

The singleton is encapsulated in a new OSGi "WebPublishingService?" as provided by the DM core.
The plugins register their resources through this service.

Not fully functional.

See ticket 272.

• Changeset: d87925be910399d851ff1bdda16632aa9c8ed6d1
• URL: ​https://github.com/jri/deepamehta/commit/d87925be910399d851ff1bdda16632aa9c8ed6d1

[Jörg Richter]

Core: adapt plugin REST services (#272).

A plugin's provider classes are picked up automatically.
A plugin must no longer provide an Application class.

The Webclient's start URL has changed:

​http://localhost:8080/de.deepamehta.webclient/

The root namespace is already occupied by the Jersey root application.

IMPORTANT: To adapt to the new Webclient base URL you must reset your database.

Not fully functional.

See ticket 272.

• Changeset: d6a40ff084ba4edd37774a6f9187e99686ebc863
• URL: ​https://github.com/jri/deepamehta/commit/d6a40ff084ba4edd37774a6f9187e99686ebc863

[Jörg Richter]

Core: WebPublishing? service basically works (#272)

Plugins add their REST resources via the WebPublishing? service.
Removing resources is pending.

Plugin's "Application" classes are dropped.

See ticket 272.

• Changeset: ec06c506990eda43bc2790fd70af4f80c520220d
• URL: ​https://github.com/jri/deepamehta/commit/ec06c506990eda43bc2790fd70af4f80c520220d

[Jörg Richter]

Core: removing REST resources dynamically (#272).

Reloading a module (via hot deployment) that provides REST resources works properly now. Other modules are not affected.
So, the main problem seems to be solved now.

Secondary issues remain (e.g. reloading the core, or shutdown) and will be addressed shortly.

Hi dgf! Your intuition with the multiple Jersey servlet instances as the culprit seems to prove correct.
Thank you!

See ticket 272.

• Changeset: f49829541c0a3c9203e829690cc867360d852901
• URL: ​https://github.com/jri/deepamehta/commit/f49829541c0a3c9203e829690cc867360d852901

[Jörg Richter]

Core: dynamic un/register Jersey servlet (#272).

Once all REST resources are removed the Jersey servlet is unregistered.
As soon as resources are added the Jersey servlet is registered again.
So, actions like reloading the DM core or shutdown the platform perform properly.

Note: there is still an issue with reloading the Webclient module but this is not REST related and needs to be addressed in another ticket.

See ticket 272.

• Changeset: b0f31852d78b883d21138f024b4c3864b6d9e6cc
• URL: ​https://github.com/jri/deepamehta/commit/b0f31852d78b883d21138f024b4c3864b6d9e6cc

[Jörg Richter]

Core: abolish resource config properties (#272).

2 properties are no longer required to be set in plugin.properties:

"restResourcesNamespace" -- now set in the plugin class's @Path anotation
"webResourcesNamespace" -- not configurable anymore

See ticket 272.

• Changeset: a39e39c19b1d0c46eec8f9abe054d7d2255b5a48
• URL: ​https://github.com/jri/deepamehta/commit/a39e39c19b1d0c46eec8f9abe054d7d2255b5a48

[Jörg Richter]

Core: plugins doesn't consume HttpService? (#272).

Instead they are register their web resources through the WebPublishingService?.

WebPublishingService? provides 2 new methods:

WebResources addWebResources(Bundle bundle, String uriNamespace)
void removeWebResources(WebResources webResources)

All web publishing features are encapsulated in the WebPublishingService?.

See ticket 272.

• Changeset: e47c8507aba7db17271d3ab3a6144944fdbdd522
• URL: ​https://github.com/jri/deepamehta/commit/e47c8507aba7db17271d3ab3a6144944fdbdd522

[dgf]

what a run - great! a hotdeploy now works as expected and the implementation looks more compatible to the OSGi especially Service design. it feels always rigth to me if some configurations can be eleminated in favour of convention like the provider package.

[jri]

Thanks for the kind comment :-)
In fact it was you who gave the trigger for dealing with that nasty problem now and who delivered the basic idea for solution.

Meanwhile I have a clue what might be the original problem cause: The Plugin base class's dependeny on the ServletContainer? class. Once the latter is unloaded by Felix all bundles depending on it are stopped. That is *all* bundles (because all depend on Plugin). Now, with the new design, Plugin has no Jersey dependency at all, only the Core module has.
You can see the same bundle stopping behavior when redeploying the Core module: Felix stops all bundles because all depend on Core classes (at least on Plugin as the OSGi Activator). But in this case this is exactly the intended behavior.

I feel happy with the new implementation too.
Thanks for good cooperation!

[Jörg Richter]

Core: use a Jersey servlet singleton (#272).

We hope this will solve the oddities with module hot deployment.

The singleton is encapsulated in a new OSGi "WebPublishingService?" as provided by the DM core.
The plugins register their resources through this service.

Not fully functional.

See ticket 272.

• Changeset: d87925be910399d851ff1bdda16632aa9c8ed6d1
• URL: ​https://github.com/jri/deepamehta/commit/d87925be910399d851ff1bdda16632aa9c8ed6d1

[Jörg Richter]

Core: adapt plugin REST services (#272).

A plugin's provider classes are picked up automatically.
A plugin must no longer provide an Application class.

The Webclient's start URL has changed:

​http://localhost:8080/de.deepamehta.webclient/

The root namespace is already occupied by the Jersey root application.

IMPORTANT: To adapt to the new Webclient base URL you must reset your database.

Not fully functional.

See ticket 272.

• Changeset: d6a40ff084ba4edd37774a6f9187e99686ebc863
• URL: ​https://github.com/jri/deepamehta/commit/d6a40ff084ba4edd37774a6f9187e99686ebc863

[Jörg Richter]

Core: WebPublishing? service basically works (#272)

Plugins add their REST resources via the WebPublishing? service.
Removing resources is pending.

Plugin's "Application" classes are dropped.

See ticket 272.

• Changeset: ec06c506990eda43bc2790fd70af4f80c520220d
• URL: ​https://github.com/jri/deepamehta/commit/ec06c506990eda43bc2790fd70af4f80c520220d

[Jörg Richter]

Core: removing REST resources dynamically (#272).

Reloading a module (via hot deployment) that provides REST resources works properly now. Other modules are not affected.
So, the main problem seems to be solved now.

Secondary issues remain (e.g. reloading the core, or shutdown) and will be addressed shortly.

Hi dgf! Your intuition with the multiple Jersey servlet instances as the culprit seems to prove correct.
Thank you!

See ticket 272.

• Changeset: f49829541c0a3c9203e829690cc867360d852901
• URL: ​https://github.com/jri/deepamehta/commit/f49829541c0a3c9203e829690cc867360d852901

[Jörg Richter]

Core: dynamic un/register Jersey servlet (#272).

Once all REST resources are removed the Jersey servlet is unregistered.
As soon as resources are added the Jersey servlet is registered again.
So, actions like reloading the DM core or shutdown the platform perform properly.

Note: there is still an issue with reloading the Webclient module but this is not REST related and needs to be addressed in another ticket.

See ticket 272.

• Changeset: b0f31852d78b883d21138f024b4c3864b6d9e6cc
• URL: ​https://github.com/jri/deepamehta/commit/b0f31852d78b883d21138f024b4c3864b6d9e6cc

[Jörg Richter]

Core: abolish resource config properties (#272).

2 properties are no longer required to be set in plugin.properties:

"restResourcesNamespace" -- now set in the plugin class's @Path anotation
"webResourcesNamespace" -- not configurable anymore

See ticket 272.

• Changeset: a39e39c19b1d0c46eec8f9abe054d7d2255b5a48
• URL: ​https://github.com/jri/deepamehta/commit/a39e39c19b1d0c46eec8f9abe054d7d2255b5a48

[Jörg Richter]

Core: plugins doesn't consume HttpService? (#272).

Instead they are register their web resources through the WebPublishingService?.

WebPublishingService? provides 2 new methods:

WebResources addWebResources(Bundle bundle, String uriNamespace)
void removeWebResources(WebResources webResources)

All web publishing features are encapsulated in the WebPublishingService?.

See ticket 272.

• Changeset: e47c8507aba7db17271d3ab3a6144944fdbdd522
• URL: ​https://github.com/jri/deepamehta/commit/e47c8507aba7db17271d3ab3a6144944fdbdd522