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

Chia-Ping Tsai commented on HBASE-18951:
----------------------------------------

bq. This is intentional as the title here is ‘Remove nullable parameters’. It 
is not recommend to use Optional as parameter so...
Got it.

BTW, all null check used in async client module pass the error message to the 
{{Preconditions.checkNotNull}}, but this patch doesn't. Is it intentional?

> Use Builder pattern to remove nullable parameters for checkAndXXX methods in 
> RawAsyncTable/AsyncTable interface
> ---------------------------------------------------------------------------------------------------------------
>
>                 Key: HBASE-18951
>                 URL: https://issues.apache.org/jira/browse/HBASE-18951
>             Project: HBase
>          Issue Type: Sub-task
>          Components: Client
>            Reporter: Duo Zhang
>            Assignee: Duo Zhang
>            Priority: Blocker
>             Fix For: 2.0.0-beta-1
>
>         Attachments: HBASE-18951.patch
>
>
> As Optional is not supposed to be used as method parameters but we do not 
> want nullable parameters.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to