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

Add drain-before-shutdown behavior (fixes #24) #57

Closed

Conversation

jchambers
Copy link
Owner

This changes the PushManager's shutdown behavior to drain the internal retry queue before gracefully closing connections.

This is the same as #50, but now with less crazy. It also depends on #51 (which depends on #47).

This is the same as #52, but now against the threadless_connections branch.

jchambers and others added 21 commits February 22, 2014 20:36

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.

Verified

This commit was signed with the committer’s verified signature. The key has expired.
doubledup David Dunn
…we'll just loop back through if a connection is added while we're shutting down other connections).
…ore_shutdown_take2

Conflicts:
	src/main/java/com/relayrides/pushy/apns/PushManager.java
…ore_shutdown_take2

Conflicts:
	src/main/java/com/relayrides/pushy/apns/PushManager.java
…ore_shutdown_take2

Conflicts:
	src/main/java/com/relayrides/pushy/apns/PushManager.java
…ore_shutdown_take2

Conflicts:
	src/main/java/com/relayrides/pushy/apns/ApnsConnection.java
@jchambers jchambers added this to the v0.3 milestone Mar 26, 2014
@jchambers
Copy link
Owner Author

Closing this in favor of a simpler (I hope) approach.

@jchambers jchambers closed this Apr 2, 2014
@jchambers jchambers deleted the drain_retry_queue_before_shutdown_take2 branch April 3, 2014 22:32
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.

None yet

1 participant