[
https://issues.apache.org/jira/browse/HADOOP-7594?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13095057#comment-13095057
]
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
