OF-2951: Async stanza delivery #2653
Open
+68
−36
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Allow a packet to be delivered asynchronously. As the act of delivering typically does not yield a result, invokers typically do not need to block for this to complete. Changes in this commit facilitate an invoker to asynchronously handle any exceptions thrown during delivery.
When probing presence of entities on remote domains, do not block for the server-to-server session to be established. This intends to speed up the process of starting up a client (which now no longer needs to wait for outbound s2s connections to be established).
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Adds a facility to allow for non-blocking stanza delivery.
The new facility is used when probing presence of entities on remote domains (as that can slow down a client that's connecting to Openfire).
Potentially this could/should be used in other places too (as the act of delivery doesn't return a result, and thus isn't expected to be needing a blocking operation), but I thought it best to take baby steps here.