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

Yonatan Gottesman commented on OMID-110:
----------------------------------------

Hi,

I finished the implementation but im having some trouble with the coprocesor 
tests.

It seems like the coprocessors are installed but the functions are not called 
(preGet...)

Hope it will be done in a 1-2 days

> Support HBase 2.0
> -----------------
>
>                 Key: OMID-110
>                 URL: https://issues.apache.org/jira/browse/OMID-110
>             Project: Apache Omid
>          Issue Type: Improvement
>            Reporter: James Taylor
>            Assignee: Yonatan Gottesman
>            Priority: Major
>         Attachments: OMID-110_wip1.patch
>
>
> Once OMID-107 is committed (which replaced all deprecated methods), the only 
> things I've identified that need to change are:
>  * Instantiation of CellComparator should be done through HBaseShims since 
> it's different between HBase 1.x and 2.x. In HBase 2.0, CellComparator was 
> changed to an interface. Another alternative would be to write our own 
> CellComparator.
> {code:java}
> public static SortedMap<Cell, Optional<Cell>> 
> mapCellsToShadowCells(List<Cell> cells) {
> // Move CellComparator to HBaseSims for 2.0 support
> // Need to access through CellComparatorImpl.COMPARATOR
> SortedMap<Cell, Optional<Cell>> cellToShadowCellMap
> = new TreeMap<Cell, Optional<Cell>>(new CellComparator());{code}
> The HBase 2.0 version would use CellComparatorImpl.COMPARATOR instead of 
> instantiating a CellComparator.
>  * Add createTable method to HBaseShims. Instead of HTableDescriptor, HBase 
> 2.0 uses TableDescriptor and TableDescriptorBuilder. 
> [~chrajeshbab...@gmail.com] and [~an...@apache.org] - do you know why 
> existing admin.createTable(HTableDescriptor) wouldn't work? HTableDescriptor 
> implements TableDescriptor.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to