Anoop Sam John commented on HBASE-18446:

Regarding HBASE-6224 and load files in CP itself.
I don't think any one would be doing that.  At that time we had a design of all 
pre hooks can do a bypass on the Context so the actual op wont be done. Similar 
way this was also done I guess.  Because u can see in RSRpcServices
if (region.getCoprocessorHost() != null) {
          bypass = region.getCoprocessorHost().preBulkLoadHFile(familyPaths);
        try {
          if (!bypass) {
            map = region.bulkLoadHFiles(familyPaths, request.getAssignSeqNum(), 
            if (map != null) {
              loaded = true;
      return builder.build();
When bypass is done, we set loaded as false only. The return to the called code 
will indicate the bulk load being NOT done. The intent of that old jira was to 
add the pre hook so that AC can check and if failed it would throw Exception 
only.  May be we can consider this op as some thing on which bypass is not 
possible!  (Like preClose)

> Mark StoreFileScanner as IA.Private
> -----------------------------------
>                 Key: HBASE-18446
>                 URL: https://issues.apache.org/jira/browse/HBASE-18446
>             Project: HBase
>          Issue Type: Sub-task
>          Components: Coprocessors
>            Reporter: Duo Zhang
>             Fix For: 2.0.0, 3.0.0, 2.0.0-alpha-2
> Do not see any reason why it is marked as IA.LimitedPrivate. It is not 
> referenced in any CPs.

This message was sent by Atlassian JIRA

Reply via email to