shiro-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <>
Subject [jira] [Commented] (SHIRO-566) CollectionUtils should use Collections wrappers of arrays if possible
Date Wed, 29 Jun 2016 22:08:13 GMT


ASF GitHub Bot commented on SHIRO-566:

Github user asfgit closed the pull request at:

> CollectionUtils should use Collections wrappers of arrays if possible
> ---------------------------------------------------------------------
>                 Key: SHIRO-566
>                 URL:
>             Project: Shiro
>          Issue Type: Improvement
>          Components: Authorization (access control) 
>    Affects Versions: 1.2.4
>         Environment: Any
>            Reporter: Matt Bishop
>            Priority: Trivial
> org.apache.shiro.util.CollectionUtils.asSet() and .asList() could use Collections wrappers
for the passed-in elements. These are smaller and faster than copying the contents into other
> The editability of the returned Collections is not required as currently it will return
either an unmodifiable emptyList() / emptySet() or a modifiable ArrayList / LinkedHashSet.
Ideally it would wrap the LinkedHashSet in unmodifiableSet and declare the immutability of
the returned collections in the docs.
> In my use case for Shiro we create about 6000 - 7000 WildcardPermission objects per second.
The great majority of them have a a list of parts with single subparts and thus don't need
a proper LinkedHashSet created. This change improve the performance of our system by 10% under
heavy load, mostly due to the reduced GC of temporary LinkedHashSet objects. 

This message was sent by Atlassian JIRA

View raw message