[ 
https://issues.apache.org/jira/browse/HBASE-4649?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13134233#comment-13134233
 ] 

Ted Yu commented on HBASE-4649:
-------------------------------

There are a few blank lines in TestHRegionServerBulkLoad.
There seems to be some overlap between the test and 
TestLoadIncrementalHFiles.java

Since the patch is part of fix for HBASE-4552, I wonder what the plan is to 
modify LoadIncrementalHFiles where the list of <family, path> pairs would be 
built by multiple threads. In other words, what data structure would be used to 
accommodate concurrency, assuming the current form of LoadIncrementalHFiles is 
kept ?

The above is related to my comment on HBASE-4552 @ 12/Oct/11 19:00.
                
> Add atomic bulk load function to region server
> ----------------------------------------------
>
>                 Key: HBASE-4649
>                 URL: https://issues.apache.org/jira/browse/HBASE-4649
>             Project: HBase
>          Issue Type: Sub-task
>          Components: regionserver
>    Affects Versions: 0.90.4, 0.92.0
>            Reporter: Jonathan Hsieh
>            Assignee: Jonathan Hsieh
>         Attachments: 
> 0001-HBASE-4649-Add-atomic-bulk-load-function-to-region-s.patch
>
>
> Add a method that atomically bulk load multiple hfiles.  Row atomicity 
> guarantees for multi-column family rows require this functionality.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to