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

Keith Turner commented on ACCUMULO-1009:
----------------------------------------

Took a quick look at the patch.   The patch adds a lot of constructors to 
ZooKeeperInstance.   I'm thinking of just adding the following constructor :

{code:java}
 /**
  * @param config provides all information needed to connect to an Accumulo 
instance.   @see ClientConfiguration 
  */
 public ZooKeeperInstance(AccumuloConfiguration config){}
{code}

This would allow us maximum flexibility in the future w/o clutering the API as 
new features are added.  The zookeepers, instance name, wether or not use ssl, 
etc. would be obtained from the config.  This constructor would be the way to 
enable ssl.

There is some code that pushes the client address and port across threads so 
its shows up in list scans, do you know if this is still working with the 
changes you made?  

                
> Support encryption over the wire
> --------------------------------
>
>                 Key: ACCUMULO-1009
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-1009
>             Project: Accumulo
>          Issue Type: New Feature
>            Reporter: Keith Turner
>            Assignee: Michael Berman
>             Fix For: 1.6.0
>
>         Attachments: ACCUMULO-1009_thriftSsl.patch
>
>
> Need to support encryption between ACCUMULO clients and servers.  Also need 
> to encrypt communications between server and servers.   
> Basically need to make it possible for users to enable SSL+thrift.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to