[ 
https://issues.apache.org/jira/browse/HBASE-5732?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Devaraj Das updated HBASE-5732:
-------------------------------

    Attachment: rpcengine-merge.3.patch

Attached is the patch with the review comments addressed (except the ones to do 
with introduction of new 'white-spaces'). This is still under test but I 
thought I'll update the jira with what I have currently.. 

Stack, i have retained User.java class upon feedback from Andrew though that 
supports only the 1.0++ Hadoop's UserGroupInformation API presently (and that 
is compatible with hadoop-2.0, etc. from HBase's usage point of view). 

I have moved the Java files inside the $root/security/src/* directory to src/*, 
and removed the security profile from the pom.xml file. 

Stack, on the copy comment you made on saslServer.wrap(), that's how Sasl API 
works. I don't think there is a way around it..

No there are no tests introduced by this patch. There're quite a few tests 
already for security. I'll ensure they continue to work (smile). Also I'll test 
the patch manually on a real cluster.

Thanks, folks for the reviews!
                
> Remove the SecureRPCEngine and merge the security-related logic in the core 
> engine
> ----------------------------------------------------------------------------------
>
>                 Key: HBASE-5732
>                 URL: https://issues.apache.org/jira/browse/HBASE-5732
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Devaraj Das
>            Assignee: Devaraj Das
>         Attachments: rpcengine-merge.3.patch, rpcengine-merge.patch
>
>
> Remove the SecureRPCEngine and merge the security-related logic in the core 
> engine. Follow up to HBASE-5727.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to