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

ASF GitHub Bot commented on STORM-1470:
---------------------------------------

Github user ptgoetz commented on the pull request:

    https://github.com/apache/storm/pull/1011#issuecomment-171796975
  
    Personally, I don't think we need separate pull requests for each branch 
yet (master and 1.x have not diverged enough to cause conflicts), as it kind of 
confuses things in terms of +1s and comments. It might be easier to add a note 
requesting that it also be applied to another branch, e.g. "I would also like 
to see this applied to the X branch."
    
    If for some reason the patch couldn't be applied to the other branch(es), 
the committer doing the merge could either resolve the conflicts or request an 
additional pull request against the conflicted branch.
    
    Thoughts?


> org.apache.hadoop:hadoop-auth not shaded, breaks spnego authentication
> ----------------------------------------------------------------------
>
>                 Key: STORM-1470
>                 URL: https://issues.apache.org/jira/browse/STORM-1470
>             Project: Apache Storm
>          Issue Type: Bug
>          Components: storm-core
>    Affects Versions: 1.0.0, 2.0.0
>            Reporter: Derek Dagit
>            Assignee: Derek Dagit
>            Priority: Blocker
>
> {noformat}
> 2016-01-12 20:07:45.642 o.a.s.s.o.e.j.s.ServletHandler [WARN] Error for 
> /favicon.ico
> java.lang.NoClassDefFoundError: org/apache/commons/codec/binary/Base64
>         at 
> org.apache.hadoop.security.authentication.server.KerberosAuthenticationHandler.authenticate(KerberosAuthenticationHandler.java:343)
>         at 
> org.apache.hadoop.security.authentication.server.AuthenticationFilter.doFilter(AuthenticationFilter.java:519)
>         at 
> org.apache.storm.shade.org.eclipse.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1291)
>         at 
> org.apache.storm.shade.org.eclipse.jetty.servlets.CrossOriginFilter.handle(CrossOriginFilter.java:247)
>         at 
> org.apache.storm.shade.org.eclipse.jetty.servlets.CrossOriginFilter.doFilter(CrossOriginFilter.java:210)
>         at 
> org.apache.storm.shade.org.eclipse.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1291)
>         at 
> org.apache.storm.shade.org.eclipse.jetty.servlet.ServletHandler.doHandle(ServletHandler.java:443)
>         at 
> org.apache.storm.shade.org.eclipse.jetty.server.handler.ContextHandler.doHandle(ContextHandler.java:1044)
>         at 
> org.apache.storm.shade.org.eclipse.jetty.servlet.ServletHandler.doScope(ServletHandler.java:372)
>         at 
> org.apache.storm.shade.org.eclipse.jetty.server.handler.ContextHandler.doScope(ContextHandler.java:978)
>         at 
> org.apache.storm.shade.org.eclipse.jetty.server.handler.ScopedHandler.handle(ScopedHandler.java:135)
>         at 
> org.apache.storm.shade.org.eclipse.jetty.server.handler.HandlerWrapper.handle(HandlerWrapper.java:116)
>         at 
> org.apache.storm.shade.org.eclipse.jetty.server.Server.handle(Server.java:369)
>         at 
> org.apache.storm.shade.org.eclipse.jetty.server.AbstractHttpConnection.handleRequest(AbstractHttpConnection.java:486)
>         at 
> org.apache.storm.shade.org.eclipse.jetty.server.AbstractHttpConnection.headerComplete(AbstractHttpConnection.java:933)
>         at 
> org.apache.storm.shade.org.eclipse.jetty.server.AbstractHttpConnection$RequestHandler.headerComplete(AbstractHttpConnection.java:995)
>         at 
> org.apache.storm.shade.org.eclipse.jetty.http.HttpParser.parseNext(HttpParser.java:644)
>         at 
> org.apache.storm.shade.org.eclipse.jetty.http.HttpParser.parseAvailable(HttpParser.java:235)
>         at 
> org.apache.storm.shade.org.eclipse.jetty.server.AsyncHttpConnection.handle(AsyncHttpConnection.java:82)
>         at 
> org.apache.storm.shade.org.eclipse.jetty.io.nio.SelectChannelEndPoint.handle(SelectChannelEndPoint.java:668)
>         at 
> org.apache.storm.shade.org.eclipse.jetty.io.nio.SelectChannelEndPoint$1.run(SelectChannelEndPoint.java:52)
>         at 
> org.apache.storm.shade.org.eclipse.jetty.util.thread.QueuedThreadPool.runJob(QueuedThreadPool.java:608)
>         at 
> org.apache.storm.shade.org.eclipse.jetty.util.thread.QueuedThreadPool$3.run(QueuedThreadPool.java:543)
>         at java.lang.Thread.run(Thread.java:745)
> {noformat}
> We already shade commons-codec:commons-codec, but we don't apply that shading 
> to org.apache.hadoop:hadoop-auth.



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

Reply via email to