[jira] Commented: (HDFS-823) In Checkpointer the getImage servlet is added to public rather than internal servlet list
[ https://issues.apache.org/jira/browse/HDFS-823?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12799013#action_12799013 ] Hudson commented on HDFS-823: - Integrated in Hdfs-Patch-h2.grid.sp2.yahoo.net #94 (See [http://hudson.zones.apache.org/hudson/job/Hdfs-Patch-h2.grid.sp2.yahoo.net/94/]) In Checkpointer the getImage servlet is added to public rather than internal servlet list - Key: HDFS-823 URL: https://issues.apache.org/jira/browse/HDFS-823 Project: Hadoop HDFS Issue Type: Bug Components: name-node Affects Versions: 0.21.0, 0.22.0 Reporter: Jakob Homan Assignee: Jakob Homan Fix For: 0.21.0, 0.22.0 Attachments: HDFS-823.patch Checkpointer.java:99 {code} httpServer.addServlet(getimage, /getimage, GetImageServlet.class);{code} This should be addInternalServlet, as it is for Namenode to ensure this servlet does not get filtered. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Commented: (HDFS-823) In Checkpointer the getImage servlet is added to public rather than internal servlet list
[ https://issues.apache.org/jira/browse/HDFS-823?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12791623#action_12791623 ] Hudson commented on HDFS-823: - Integrated in Hdfs-Patch-h5.grid.sp2.yahoo.net #148 (See [http://hudson.zones.apache.org/hudson/job/Hdfs-Patch-h5.grid.sp2.yahoo.net/148/]) In Checkpointer the getImage servlet is added to public rather than internal servlet list - Key: HDFS-823 URL: https://issues.apache.org/jira/browse/HDFS-823 Project: Hadoop HDFS Issue Type: Bug Components: name-node Affects Versions: 0.21.0, 0.22.0 Reporter: Jakob Homan Assignee: Jakob Homan Fix For: 0.21.0, 0.22.0 Attachments: HDFS-823.patch Checkpointer.java:99 {code} httpServer.addServlet(getimage, /getimage, GetImageServlet.class);{code} This should be addInternalServlet, as it is for Namenode to ensure this servlet does not get filtered. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Commented: (HDFS-823) In Checkpointer the getImage servlet is added to public rather than internal servlet list
[ https://issues.apache.org/jira/browse/HDFS-823?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12789009#action_12789009 ] Jakob Homan commented on HDFS-823: -- Looks like Hudson is falling down on the job again. Manual test-patch: {noformat}[exec] -1 overall. [exec] [exec] +1 @author. The patch does not contain any @author tags. [exec] [exec] -1 tests included. The patch doesn't appear to include any new or modified tests. [exec] Please justify why no new tests are needed for this patch. [exec] Also please list what manual steps were performed to verify this patch. [exec] [exec] +1 javadoc. The javadoc tool did not generate any warning messages. [exec] [exec] +1 javac. The applied patch does not increase the total number of javac compiler warnings. [exec] [exec] +1 findbugs. The patch does not introduce any new Findbugs warnings. [exec] [exec] +1 release audit. The applied patch does not increase the total number of release audit warnings.{noformat} No tests as explained above. Tests all pass locally except known-bad TestHDFSTrash. Will commit to trunk and 21. In Checkpointer the getImage servlet is added to public rather than internal servlet list - Key: HDFS-823 URL: https://issues.apache.org/jira/browse/HDFS-823 Project: Hadoop HDFS Issue Type: Bug Reporter: Jakob Homan Assignee: Jakob Homan Attachments: HDFS-823.patch Checkpointer.java:99 {code} httpServer.addServlet(getimage, /getimage, GetImageServlet.class);{code} This should be addInternalServlet, as it is for Namenode to ensure this servlet does not get filtered. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.