[
https://issues.apache.org/jira/browse/HBASE-18446?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16116177#comment-16116177
]
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
{code}
if (region.getCoprocessorHost() != null) {
bypass = region.getCoprocessorHost().preBulkLoadHFile(familyPaths);
}
try {
if (!bypass) {
map = region.bulkLoadHFiles(familyPaths, request.getAssignSeqNum(),
null,
request.getCopyFile());
if (map != null) {
loaded = true;
}
}
...
builder.setLoaded(loaded);
return builder.build();
{code}
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
(v6.4.14#64029)
