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

Appy commented on HBASE-18878:
------------------------------

Looks like Optional is not supposed to be used for parameters. (I started 
reading around when my IDE showed a little notice on {{Optional<User> user}}.)
- 
https://stackoverflow.com/questions/31922866/why-should-java-8s-optional-not-be-used-in-arguments
- http://dolszewski.com/java/java-8-optional-use-cases/

Let me put up a subset of [~Apache9]'s patch.


> Use Optional instead of Nullable annotation in ObserverContext to better 
> describe that the caller may not be presented
> ----------------------------------------------------------------------------------------------------------------------
>
>                 Key: HBASE-18878
>                 URL: https://issues.apache.org/jira/browse/HBASE-18878
>             Project: HBase
>          Issue Type: Sub-task
>          Components: Coprocessors
>            Reporter: Duo Zhang
>            Assignee: Duo Zhang
>             Fix For: 2.0.0-alpha-4
>
>         Attachments: HBASE-18878.patch, HBASE-18878-v1.patch
>
>
> I've already done lots of Nullable to Optional change when purging the 
> interfaces for CP. This is a big one so open a separated issue for it.



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

Reply via email to