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

Christopher Tubbs commented on ACCUMULO-1009:
---------------------------------------------

I won't argue that the JSSE properties *must* come from the system properties, 
but you're shooting yourself in the foot if you're rewriting custom socket code 
to accept them anywhere else, and somebody's going to have to maintain that 
additional code.

At the very least, I still think using the JSSE properties are essential... I 
just won't argue they have to be set as system properties (which are global). 
They could just as well come from a client config file. What I wouldn't want to 
see is custom config key names that users have to relearn, but essentially map 
to the same functionality as the JSSE ones they're already familiar with. At 
that point, it'd be like adding new words to a dictionary, whose definitions 
are "see some_other_word".

As for the client configuration, I don't mind them crossing paths at this 
point... but, if we're going to do the additional effort in the future to 
better scope configuration to the objects they configure at some point (and I 
hope we are), I don't want to be stuck with the paths being crossed in the API. 
For that reason, I would not put the static method on AccumuloConfiguration. 
I'd keep it a completely separate thing (in fact, I'd probably keep it a simple 
commons-configuration Configuration object). If we need reuse, I'd tie it 
underneath, so when we do the effort of better scoping configuration in the 
future, it won't disrupt the API, where users interact.
                
> 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