hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "stack (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-18926) Cleanup Optional<T> from method params
Date Fri, 06 Oct 2017 16:40:02 GMT

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

stack commented on HBASE-18926:
-------------------------------

bq. We can change api after beta1. So either way, should be okay to fix in beta1, no?

You meant we can *NOT* change API after API [~appy]?


> Cleanup Optional<T> from method params
> --------------------------------------
>
>                 Key: HBASE-18926
>                 URL: https://issues.apache.org/jira/browse/HBASE-18926
>             Project: HBase
>          Issue Type: Umbrella
>            Reporter: Appy
>            Assignee: Duo Zhang
>
> ref: Discussion on HBASE-18878
> tldr: Optional is not supposed to be used in method params or class fields (http://dolszewski.com/java/java-8-optional-use-cases/).
We are doing that in some places. Let's revert that.



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

Mime
View raw message