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

Ted Yu commented on HBASE-21194:
--------------------------------

{code}
+      assertTrue("if the assertion fails, the message should give the mob row 
count (0)",
{code}
to:
{code}
+      assertTrue("The mob row count is 0 but should be > 0",
{code}
Please check the indentation level in several places:
e.g.
{code}
+      g = new Get(row2);
+      r = t2.get(g);
+      assertEquals(0, r.size());
+
+      } finally {
+          TEST_UTIL.deleteTable(tableName1);
{code}
The body of try should be indented by 2 space.
Same with body of finally.

We're very close.

> Add TestCopyTable which exercises MOB feature
> ---------------------------------------------
>
>                 Key: HBASE-21194
>                 URL: https://issues.apache.org/jira/browse/HBASE-21194
>             Project: HBase
>          Issue Type: Test
>            Reporter: Ted Yu
>            Assignee: Artem Ervits
>            Priority: Minor
>              Labels: mob
>         Attachments: HBASE-21194.v01.patch, HBASE-21194.v02.patch, 
> HBASE-21194.v03.patch, HBASE-21194.v06.patch
>
>
> Currently TestCopyTable doesn't cover table(s) with MOB feature enabled.
> We should add variant that enables MOB on the table being copied and verify 
> that MOB content is copied correctly.



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

Reply via email to