-
Notifications
You must be signed in to change notification settings - Fork 8.9k
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
HADOOP-17621. hadoop-auth to remove jetty-server dependency. #2865
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 pending Jenkins.
I'm also waiting for the internal integration tests to pass. |
💔 -1 overall
This message was automatically generated. |
The PR passed our internal integration tests. |
The javac warning is expected. I intend to use this deprecated API to achieve the goal. |
Agreed. |
Reviewed-by: Akira Ajisaka <aajisaka@apache.org> (cherry picked from commit dac60b8)
…OP-17625 to update Jetty to 9.4.39. (#2879) * HADOOP-17371. Bump Jetty to the latest version 9.4.34. Contributed by Wei-Chiu Chuang. (#2453) (cherry picked from commit 66ee0a6) (cherry picked from commit 6340ac8) Conflicts: hadoop-client-modules/hadoop-client-minicluster/pom.xml Change-Id: I673ac136922740cb1d426cb9593aa1bd3e9acd32 * HADOOP-17621. hadoop-auth to remove jetty-server dependency. (#2865) Reviewed-by: Akira Ajisaka <aajisaka@apache.org> (cherry picked from commit dac60b8) (cherry picked from commit 1110b03) * HADOOP-17625. Update to Jetty 9.4.39. (#2870) Reviewed-by: cxorm <lianp964@gmail.com> (cherry picked from commit 6040e86) (cherry picked from commit 7f75355)
…OP-17625 to update Jetty to 9.4.39. (apache#2879) * HADOOP-17371. Bump Jetty to the latest version 9.4.34. Contributed by Wei-Chiu Chuang. (apache#2453) (cherry picked from commit 66ee0a6) (cherry picked from commit 6340ac8) Conflicts: hadoop-client-modules/hadoop-client-minicluster/pom.xml Change-Id: I673ac136922740cb1d426cb9593aa1bd3e9acd32 * HADOOP-17621. hadoop-auth to remove jetty-server dependency. (apache#2865) Reviewed-by: Akira Ajisaka <aajisaka@apache.org> (cherry picked from commit dac60b8) (cherry picked from commit 1110b03) * HADOOP-17625. Update to Jetty 9.4.39. (apache#2870) Reviewed-by: cxorm <lianp964@gmail.com> (cherry picked from commit 6040e86) (cherry picked from commit 7f75355) (cherry picked from commit 8ff61f9) Conflicts: hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/ImageServlet.java Change-Id: I465f6003b6f4c5df9c41c83eac3738bac56403e1
…OP-17625 to update Jetty to 9.4.39. (#2879) (#2935) * HADOOP-17371. Bump Jetty to the latest version 9.4.34. Contributed by Wei-Chiu Chuang. (#2453) (cherry picked from commit 66ee0a6) (cherry picked from commit 6340ac8) Conflicts: hadoop-client-modules/hadoop-client-minicluster/pom.xml Change-Id: I673ac136922740cb1d426cb9593aa1bd3e9acd32 * HADOOP-17621. hadoop-auth to remove jetty-server dependency. (#2865) Reviewed-by: Akira Ajisaka <aajisaka@apache.org> (cherry picked from commit dac60b8) (cherry picked from commit 1110b03) * HADOOP-17625. Update to Jetty 9.4.39. (#2870) Reviewed-by: cxorm <lianp964@gmail.com> (cherry picked from commit 6040e86) (cherry picked from commit 7f75355) (cherry picked from commit 8ff61f9) Conflicts: hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/ImageServlet.java Change-Id: I465f6003b6f4c5df9c41c83eac3738bac56403e1
…2865) Reviewed-by: Akira Ajisaka <aajisaka@apache.org>
Use setStatus() API defined in J2EE's HttpServletResponse. Jetty's Response#setStatus() calls setStatusWithReason() so it's all the same for Jetty.
NOTICE
Please create an issue in ASF JIRA before opening a pull request,
and you need to set the title of the pull request which starts with
the corresponding JIRA issue number. (e.g. HADOOP-XXXXX. Fix a typo in YYY.)
For more details, please see https://cwiki.apache.org/confluence/display/HADOOP/How+To+Contribute