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

ASF GitHub Bot commented on DRILL-5431:
---------------------------------------

Github user sohami commented on a diff in the pull request:

    https://github.com/apache/drill/pull/950#discussion_r140401069
  
    --- Diff: exec/rpc/src/main/java/org/apache/drill/exec/rpc/BasicServer.java 
---
    @@ -203,6 +226,9 @@ public void close() throws IOException {
           if (elapsed > 500) {
             logger.info("closed eventLoopGroup " + eventLoopGroup + " in " + 
elapsed + " ms");
           }
    +      if(isSslEnabled()) {
    --- End diff --
    
    `if(isSslEnabled())` check not required here and in `BasicClient` since 
`closeSSL` is taking care of it.


> Support SSL
> -----------
>
>                 Key: DRILL-5431
>                 URL: https://issues.apache.org/jira/browse/DRILL-5431
>             Project: Apache Drill
>          Issue Type: New Feature
>          Components: Client - Java, Client - ODBC
>            Reporter: Sudheesh Katkam
>            Assignee: Sudheesh Katkam
>
> Support SSL between Drillbit and JDBC/ODBC drivers. Drill already supports 
> HTTPS for web traffic.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to