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

ASF GitHub Bot commented on TAJO-1286:
--------------------------------------

Github user hyunsik commented on the pull request:

    https://github.com/apache/tajo/pull/341#issuecomment-69296396
  
    +1 ship it. The patch looks good to me. 
    
    Even though TestTajoClient.testKillQuery causes one test failures, it is 
not related to this issue. Actually, I'm digging this problem. I'll fix this 
problem in another jira. Your patch seems to work correctly.  


> Remove netty dependency from tajo-jdbc
> --------------------------------------
>
>                 Key: TAJO-1286
>                 URL: https://issues.apache.org/jira/browse/TAJO-1286
>             Project: Tajo
>          Issue Type: Improvement
>          Components: JDBC
>    Affects Versions: 0.10
>            Reporter: Jihun Kang
>            Assignee: Jihun Kang
>            Priority: Minor
>
> tajo-jdbc module depends on the netty class, QueryStringDecoder. It has no 
> problem since we use the netty 3.x library, but when we use the netty 4.x or 
> above, netty http codec library includes bunches of netty modules for itself. 
> It needs to clean up dependencies from tajo jdbc driver.



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

Reply via email to