[ https://issues.apache.org/jira/browse/HBASE-17123?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15678349#comment-15678349 ]
Enis Soztutar commented on HBASE-17123: --------------------------------------- The patch looks good. We want to also deprecate the RegionObserver method in favor of the previous one. Can you please rename the {{map}} argument to something like {{finalPaths}} or {{storeFiles}}. Maybe also change familyPaths to {{stagingFamilyPaths}} to make it explicit that these are paths in the staging directory. > Add postBulkLoadHFile variant that notifies the final paths for the hfiles > -------------------------------------------------------------------------- > > Key: HBASE-17123 > URL: https://issues.apache.org/jira/browse/HBASE-17123 > Project: HBase > Issue Type: Improvement > Reporter: Ted Yu > Assignee: Ted Yu > Attachments: 17123.v1.txt, 17123.v3.txt, 17123.v4.txt > > > Currently the postBulkLoadHFile() hook passes the same familyPaths parameter > which it receives as method parameter. > See code in SecureBulkLoadManager : > {code} > loaded = region.getCoprocessorHost().postBulkLoadHFile(familyPaths, > loaded); > {code} > Meaning, the paths are not final, moved paths of the loaded hfiles. > This issue is to add a variant which notifies the final paths of the loaded > hfiles. -- This message was sent by Atlassian JIRA (v6.3.4#6332)