forked from simonjarcher/eclipse.rt.equinox.bundles
-
Notifications
You must be signed in to change notification settings - Fork 1
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
Lps 140601 #23
Open
tinatian
wants to merge
23
commits into
shuyangzhou:R4_5_maintenance
Choose a base branch
from
tinatian:LPS-140601
base: R4_5_maintenance
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Lps 140601 #23
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
Change-Id: I4181c415f1f0f6edbf8b7583694572ac66e47e80 Signed-off-by: kitlo <[email protected]>
… inner HttpSesionAdaptors. This is needed because having the ParentSessionListener as outter HttpSession attribute would cause session replication failure, since outter HttpSession belongs to appserver, which does not have classloader access to ParentSessionListener (which lives in OSGi classloader) during deserialization. Signed-off-by: shuyangzhou <[email protected]>
…that it can cause appserver session replication deserialization failures Signed-off-by: shuyangzhou <[email protected]>
…for different servlet context, or else the temp folder created will be removed when a new ProxyContext is generated.
…ing logic for the "dispatcherType == FORWARD" branch
…ributes that have been explicitly set to null
…s very rare to even have to call the expensive one.
…explicitly set to null
…ad of multiple times
…rty we already know.
…need to get all the references by the service ID."
…The underneath HttpSession might already be recycled during destroy callback, which will return null for getId() causing NPE.
Signed-off-by: Raymond Auge <[email protected]>
Signed-off-by: Raymond Auge <[email protected]>
Signed-off-by: Raymond Auge <[email protected]>
…rCustomizer during deployment Signed-off-by: Raymond Augé <[email protected]>
…rst to shortcut early.
…rCustomizer, this is to make sure we are collect by type to avoid the full service registry scanning.
… object is expected.
…g service tracker
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.
No description provided.