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

Anoop Sam John commented on HBASE-11713:
----------------------------------------

{code}
+  private static void append(Configuration conf, String property, String 
value) {
+    conf.set(property, conf.get(property) + "," + value);
+  }
{code}
Add check before adding ",". If conf already have a value then only add ","


> Adding hbase shell unit test coverage for visibility labels.
> ------------------------------------------------------------
>
>                 Key: HBASE-11713
>                 URL: https://issues.apache.org/jira/browse/HBASE-11713
>             Project: HBase
>          Issue Type: Test
>          Components: security, shell
>            Reporter: Srikanth Srungarapu
>            Assignee: Srikanth Srungarapu
>            Priority: Minor
>         Attachments: HBASE-11713.patch, HBASE-11713_v2.patch
>
>
> Adding test coverage for visibility labels to hbase shell. Also, refactoring 
> existing tests so that all the unit tests related to visibility can be found 
> in one place.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to