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

Ted Yu updated HBASE-12164:
---------------------------
    Attachment: 12164-v1.txt

Proposed patch.

Also fixes a problem Stack discovered where required field of 
SecureBulkLoadHFilesResponse isn't set in failure condition.

> Check for presence of user Id in 
> SecureBulkLoadEndpoint#secureBulkLoadHFiles() is wrong
> ---------------------------------------------------------------------------------------
>
>                 Key: HBASE-12164
>                 URL: https://issues.apache.org/jira/browse/HBASE-12164
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Ted Yu
>            Assignee: Ted Yu
>         Attachments: 12164-v1.txt
>
>
> Here is the code:
> {code}
>     if (request.getFsToken().hasIdentifier() && 
> request.getFsToken().hasPassword()) {
> {code}
> In test case, request.getFsToken().hasIdentifier() returns false, leading to 
> userToken being null.
> This would make secure bulk load unsuccessful because the body of 
> secureBulkLoadHFiles() is skipped.



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

Reply via email to