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

Ravi Prakash commented on HADOOP-10075:
---------------------------------------

Thanks Robert! 

* Why did you have to add {{ep.getPort() == -2}} in 
{{HttpServer2.Builder.build()}}? The javadoc for URI never claims to return a 
-2. Is it bleeding in from {{ServerConnector.getLocalPort()}} somehow?
* {{HttpServer2.createDefaultChannelConnector()}} has started taking a server 
as an argument. Is that really necessary? Or are we always asking to pass in an 
instance of {{Server}} to this method that we already have access to?

My notes (and not necessarily questions, but if you have more info, it'd be 
great to document here):
* 
http://www.eclipse.org/jetty/documentation/current/architecture.html#basic-architecture
 and http://www.eclipse.org/jetty/documentation/current/embedding-jetty.html 
are good pages to read up.
* {code}      conn.addFirstConnectionFactory(new 
SslConnectionFactory(sslContextFactory,
          HttpVersion.HTTP_1_1.asString()));
{code} 
http://archive.eclipse.org/jetty/9.0.0.M3/apidocs/org/eclipse/jetty/server/ssl/SslSelectChannelConnector.html
 suggests to use SelectChannelConnector . Couldn't find any documentation on 
how to do that.
* {{c.setLowResourceMaxIdleTime(10000);}} has been changed to 
{{c.setIdleTimeout(10000);}} This just means that timeout will occur after 10s 
whether or not there were more than LowResourcesConnections. I'm fine with this 
change.
* We haven't set {{c.setResolveNames(false);}}[This 
thread|https://dev.eclipse.org/mhonarc/lists/jetty-users/msg07284.html] claims 
that by default names will not be resolved, so I'm fine with this change. 
Tagging [~mshen] who had put in original arguments.
* {{addNoCacheFilter(webAppContext);}} was changed to 
{{addNoCacheFilter(logContext);}} Was this a bug earlier?
* {{addInternalServlet()}} removes existing path bindings now. I don't see what 
else we could do about it. Fine by me.


> Update jetty dependency to version 9
> ------------------------------------
>
>                 Key: HADOOP-10075
>                 URL: https://issues.apache.org/jira/browse/HADOOP-10075
>             Project: Hadoop Common
>          Issue Type: Improvement
>    Affects Versions: 2.2.0, 2.6.0
>            Reporter: Robert Rati
>            Assignee: Robert Kanter
>            Priority: Critical
>         Attachments: HADOOP-10075-002-wip.patch, HADOOP-10075.003.patch, 
> HADOOP-10075.004.patch, HADOOP-10075.005.patch, HADOOP-10075.006.patch, 
> HADOOP-10075.007.patch, HADOOP-10075.008.patch, HADOOP-10075.009.patch, 
> HADOOP-10075.patch
>
>
> Jetty6 is no longer maintained.  Update the dependency to jetty9.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org

Reply via email to