Skip to content
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

Fix for issue #75 - Graceful shutdown feature implementation #76

Closed
wants to merge 3 commits into from

Conversation

gfigiel
Copy link
Contributor

@gfigiel gfigiel commented Sep 2, 2016

Please see the implementation of graceful stoping mode implementation required for graceful shutdown feature.

As to utilize the feature the supporting RA must implement org.mobicents.slee.container.resource.GracefullyStopableResourceAdaptor interface.
Separate process shall monitor supporting RA activities state to finally shutdown the container.

@@ -22,7 +22,12 @@

package org.mobicents.slee.resource;

import java.io.Serializable;
import org.apache.log4j.Logger;
import org.mobicents.slee.container.resource.GracefullyStopableResourceAdaptor;
Copy link
Contributor

@jaimecasero jaimecasero Sep 6, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this new interface part of this PR?

It seems its not even on your repo https://github.com/gfigiel/jain-slee/tree/2.x-gracefulshutdown/container/resource/src/main/java/org/mobicents/slee/resource
maybe you missed to git add the new file...? hopefully it is safe on your workspace

Grzegorz Figiel added 2 commits September 6, 2016 14:00
@gfigiel
Copy link
Contributor Author

gfigiel commented Sep 6, 2016

Hello,
In deed I missed this interface when commiting changes - now it should be visible...
Sorry for confusion.

Br,
Grzegorz Figiel

@gfigiel gfigiel closed this Sep 8, 2016
@gfigiel gfigiel deleted the 2.x-gracefulshutdown branch September 8, 2016 12:36
@gfigiel
Copy link
Contributor Author

gfigiel commented Sep 8, 2016

Closed as test status is not certain. Branch with fixes will be recreated when tests are finished.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants