Github user ohadshacham commented on a diff in the pull request:

    https://github.com/apache/incubator-omid/pull/41#discussion_r206839768
  
    --- Diff: 
hbase-client/src/test/java/org/apache/omid/transaction/TestCellUtils.java ---
    @@ -99,11 +99,11 @@ public void testShadowCellQualifiers(byte[] 
shadowCellSuffixToTest) throws IOExc
         public void testCorrectMapingOfCellsToShadowCells() throws IOException 
{
             // Create the required data
             final byte[] validShadowCellQualifier =
    -                com.google.common.primitives.Bytes.concat(qualifier, 
SHADOW_CELL_SUFFIX);
    +                com.google.common.primitives.Bytes.concat(new byte[1], 
qualifier, SHADOW_CELL_SUFFIX);
    --- End diff --
    
    Define SHADOW_CELL_PREFIX to "new byte[1]"  and replace all the new byte[1].


---

Reply via email to