hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Xiang Li (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-17126) Expose KeyValue#checkParameters() and checkForTagsLength() to be used by other Cell implementations
Date Fri, 18 Nov 2016 15:35:58 GMT

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

Xiang Li commented on HBASE-17126:
----------------------------------

Hi [~anoop.hbase], I uploaded patch 0 for master, which includes
(1) Set access of checkParameters() to package private
(2) Move checkForTagsLength() and MAX_TAGS_LENGTH into TagUtil. Set access of checkForTagsLength()
to package private. Keep MAX_TAGS_LENGTH as private.

What about moving checkForTagsLength() and MAX_TAGS_LENGTH into CellUtil? I think TagUtil
is for single tag, while tags would fit more into CellUtil.

> Expose KeyValue#checkParameters() and checkForTagsLength() to be used by other Cell implementations
> ---------------------------------------------------------------------------------------------------
>
>                 Key: HBASE-17126
>                 URL: https://issues.apache.org/jira/browse/HBASE-17126
>             Project: HBase
>          Issue Type: Improvement
>          Components: Client, regionserver
>    Affects Versions: 2.0.0
>            Reporter: Xiang Li
>            Assignee: Xiang Li
>            Priority: Minor
>             Fix For: 2.0.0
>
>         Attachments: HBASE-17126.master.000.patch
>
>
> KeyValue has 2 useful but private functions to check input parameters, as
> - checkParameters()
> - checkForTagsLength()
> It would be great if their's access could be not private, so that they can be used by
other Cell or ExtendedCell implementations.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message