You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hbase.apache.org by "Duo Zhang (JIRA)" <ji...@apache.org> on 2019/04/01 12:51:04 UTC

[jira] [Commented] (HBASE-22108) Avoid passing null in Admin methods

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

Duo Zhang commented on HBASE-22108:
-----------------------------------

Fix the checkstyle issues. The javac warning is not introduced by this patch, can be fixed in a separated issue.

> Avoid passing null in Admin methods
> -----------------------------------
>
>                 Key: HBASE-22108
>                 URL: https://issues.apache.org/jira/browse/HBASE-22108
>             Project: HBase
>          Issue Type: Task
>          Components: Admin
>            Reporter: Duo Zhang
>            Assignee: Duo Zhang
>            Priority: Major
>             Fix For: 3.0.0, 2.2.0, 2.3.0
>
>         Attachments: HBASE-22108-v1.patch, HBASE-22108.patch
>
>
> For some methods we must pass null if we want specific behaviors, for example, move region to a random server, split region without providing an explicit split point, etc.
> We should provide methods without some parameters so user do not need to pass null.
> So in the future users do not need to guess whether a method accept null parameters, the answer is always no.



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