You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hbase.apache.org by "Ted Yu (JIRA)" <ji...@apache.org> on 2017/12/04 17:13:00 UTC

[jira] [Created] (HBASE-19417) Properly assign loaded flag in RSRpcServices#bulkLoadHFile()

Ted Yu created HBASE-19417:
------------------------------

             Summary: Properly assign loaded flag in RSRpcServices#bulkLoadHFile()
                 Key: HBASE-19417
                 URL: https://issues.apache.org/jira/browse/HBASE-19417
             Project: HBase
          Issue Type: Bug
            Reporter: Appy
            Assignee: Ted Yu


See the discussion at the tail of HBASE-17123 where Appy pointed out that the override of loaded should be placed inside else block:
{code}
      } else {
        // secure bulk load
        map = regionServer.secureBulkLoadManager.secureBulkLoadHFiles(region, request);
      }
      BulkLoadHFileResponse.Builder builder = BulkLoadHFileResponse.newBuilder();
      if (map != null) {
        loaded = true;
      }
{code}
This issue is to address the review comment.



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