hadoop-common-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Tsz Wo (Nicholas), SZE (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HADOOP-7594) Support HTTP REST in HttpServer
Date Thu, 01 Sep 2011 01:27:10 GMT

    [ https://issues.apache.org/jira/browse/HADOOP-7594?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13095058#comment-13095058
] 

Tsz Wo (Nicholas), SZE commented on HADOOP-7594:
------------------------------------------------

> ... new method violates the HttpServer abstraction which is to hide the actual server
implementations (note, there is no jetty stuff leaked out of HttpServer). ...

I don't agree that.  Firstly, the new method requires packageName and pathSpec as parameters.
 They are strings but not Jetty specific.  Secondly, there are existing public methods requiring
Jetty class as a parameter, e.g. the ctxt parameter in {{public void addContext(Context ctxt,
boolean isFiltered)}}.  So the abstraction is not as clear as suggested.

Anyway, it would be great if you would like to improve it.

> Support HTTP REST in HttpServer
> -------------------------------
>
>                 Key: HADOOP-7594
>                 URL: https://issues.apache.org/jira/browse/HADOOP-7594
>             Project: Hadoop Common
>          Issue Type: New Feature
>            Reporter: Tsz Wo (Nicholas), SZE
>            Assignee: Tsz Wo (Nicholas), SZE
>             Fix For: 0.23.0, 0.24.0
>
>         Attachments: c7594_20110830.patch, c7594_20110830b.patch, c7594_20110831.patch
>
>
> Provide an API in HttpServer for supporting HTTP REST.
> This is a part of HDFS-2284.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Mime
View raw message