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

BELUGA BEHR commented on HBASE-19487:
-------------------------------------

[~appy] With all due respect, I believe that the situation with dictate 
explicit checks with using {{emptyIfNull}}.  There are cases where using an 
explicit check can save some object instantiation, additional branching logic, 
and the like.  Sometimes it's just faster/cleaner to check the collection and 
return a static value if it's empty.

I've updated the patch to take advantage of both options when appropriate.  
Please consider.

> Remove IterablesUtil Class
> --------------------------
>
>                 Key: HBASE-19487
>                 URL: https://issues.apache.org/jira/browse/HBASE-19487
>             Project: HBase
>          Issue Type: Improvement
>          Components: hbase
>    Affects Versions: 3.0.0
>            Reporter: BELUGA BEHR
>            Assignee: BELUGA BEHR
>         Attachments: HBASE-19487.1.patch, HBASE-19487.2.patch
>
>
> Remove mostly unused and obsolete class {{IterablesUtil}}



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

Reply via email to