You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This file contains hidden or 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
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ trunk Compile Tests _
+0 🆗
mvndep
16m 18s
Maven dependency ordering for branch
+1 💚
mvninstall
19m 42s
trunk passed
+1 💚
compile
15m 41s
trunk passed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1
+1 💚
compile
14m 17s
trunk passed with JDK Private Build-1.8.0_362-8u372-gaus1-0ubuntu120.04-b09
+1 💚
checkstyle
3m 48s
trunk passed
+1 💚
mvnsite
4m 40s
trunk passed
+1 💚
javadoc
4m 12s
trunk passed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1
+1 💚
javadoc
3m 44s
trunk passed with JDK Private Build-1.8.0_362-8u372-gaus1-0ubuntu120.04-b09
+1 💚
spotbugs
7m 57s
trunk passed
+1 💚
shadedclient
21m 20s
branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+0 🆗
mvndep
0m 29s
Maven dependency ordering for patch
+1 💚
mvninstall
2m 44s
the patch passed
+1 💚
compile
14m 50s
the patch passed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1
+1 💚
javac
14m 50s
the patch passed
+1 💚
compile
14m 23s
the patch passed with JDK Private Build-1.8.0_362-8u372-gaus1-0ubuntu120.04-b09
We're closing this stale PR because it has been open for 100 days with no activity. This isn't a judgement on the merit of the PR in any way. It's just a way of keeping the PR queue manageable.
If you feel like this was a mistake, or you would like to continue working on it, please feel free to re-open it and ask for a committer to remove the stale tag and review again.
Thanks all for your contribution.
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.
Description of PR
YARN-7747 use injected GuiceFilter instances
How was this patch tested?
A basic unit test to make sure each webapp responds with 200 when a URL unique to that webapp is contacted.