-
Notifications
You must be signed in to change notification settings - Fork 28
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 68046 #42998
Lps 68046 #42998
Conversation
…ernally is an invocation chain. After completing the current arguments manipulation, it needs to move on to finish the entire chain. The only reason we have not seen a broken case so far, is because PortletPreferencesLocalServiceStagingAdvice just happened to be the last AOP advice before reaching to the real target. In case anyone changes the Spring xml to inject anthor AOP advice in between this adivce and the real target, the old code will actually bypass the new advice directly hitting the real target causing issues.
Some tests FAILED!Build Time: 2 minutes 30 seconds Base Branch:Branch Name: master Job Summary:For more details click here.Failed Jobs:
For upstream results, click here. |
ci:retest |
Pull Request was closed!This pull request was closed because the following critical batches had failed: For information as to why we automatically close out certain pull requests see this article. *If you believe this is a mistake please re-open this pull. This pull will no longer automatically close if this comment is available. |
Some tests FAILED!Build Time: 56 minutes 29 seconds Base Branch:Branch Name: master Job Summary:
For more details click here.Failed Jobs:
For upstream results, click here. |
Irrelevant failures, please reopen this pull. |
@dantewang @slnn performance check please.
CC @Preston-Crary