[jira] [Commented] (HBASE-17126) Expose KeyValue#checkParameters() and checkForTagsLength() to be used by other Cell implementations

[ https://issues.apache.org/jira/browse/HBASE-17126?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15680789#comment-15680789 ] Xiang Li commented on HBASE-17126: -- Hi Anoop, got it, thanks, also for the patient guide and review! >

[jira] [Commented] (HBASE-17126) Expose KeyValue#checkParameters() and checkForTagsLength() to be used by other Cell implementations

[ https://issues.apache.org/jira/browse/HBASE-17126?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15679759#comment-15679759 ] Hudson commented on HBASE-17126: FAILURE: Integrated in Jenkins build HBase-Trunk_matrix #1982 (See

[jira] [Commented] (HBASE-17126) Expose KeyValue#checkParameters() and checkForTagsLength() to be used by other Cell implementations

[ https://issues.apache.org/jira/browse/HBASE-17126?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15679444#comment-15679444 ] Anoop Sam John commented on HBASE-17126: +1 CellUtil is public exposed.. TagUtil is our internal

[jira] [Commented] (HBASE-17126) Expose KeyValue#checkParameters() and checkForTagsLength() to be used by other Cell implementations

[ https://issues.apache.org/jira/browse/HBASE-17126?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15679225#comment-15679225 ] Hadoop QA commented on HBASE-17126: --- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem

[jira] [Commented] (HBASE-17126) Expose KeyValue#checkParameters() and checkForTagsLength() to be used by other Cell implementations

[ https://issues.apache.org/jira/browse/HBASE-17126?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15679169#comment-15679169 ] Xiang Li commented on HBASE-17126: -- Hi [~anoop.hbase], thanks for the comments! I uploaded patch 003 to

[jira] [Commented] (HBASE-17126) Expose KeyValue#checkParameters() and checkForTagsLength() to be used by other Cell implementations

[ https://issues.apache.org/jira/browse/HBASE-17126?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15677044#comment-15677044 ] Hadoop QA commented on HBASE-17126: --- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem

[jira] [Commented] (HBASE-17126) Expose KeyValue#checkParameters() and checkForTagsLength() to be used by other Cell implementations

[ https://issues.apache.org/jira/browse/HBASE-17126?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15677023#comment-15677023 ] Anoop Sam John commented on HBASE-17126: bq.private static final int MAX_TAGS_LENGTH = (2 *

[jira] [Commented] (HBASE-17126) Expose KeyValue#checkParameters() and checkForTagsLength() to be used by other Cell implementations

[ https://issues.apache.org/jira/browse/HBASE-17126?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15676964#comment-15676964 ] Xiang Li commented on HBASE-17126: -- Hi [~anoop.hbase], I uploaded patch 0 for master, which includes (1)

[jira] [Commented] (HBASE-17126) Expose KeyValue#checkParameters() and checkForTagsLength() to be used by other Cell implementations

[ https://issues.apache.org/jira/browse/HBASE-17126?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15676740#comment-15676740 ] Xiang Li commented on HBASE-17126: -- Hi Anoop, yes, I mean to put CheckForTagsLength() into TagUtil. >

[jira] [Commented] (HBASE-17126) Expose KeyValue#checkParameters() and checkForTagsLength() to be used by other Cell implementations

[ https://issues.apache.org/jira/browse/HBASE-17126?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15676428#comment-15676428 ] Anoop Sam John commented on HBASE-17126: checkParameters() can be package private.